eduwireapps/ynotes-packages

The packages used in yNotes.

ynotes-packages's Issues

Ne pas importer les thèmes dans components

Actuellement, components importe directement les thèmes, ce qui empêche son utilisation en dehors de yNotes. Il faudrait donc passer les thèmes à la lib dans l'application.

Je suppose que le mieux c'est de passer les thèmes dans le main avec une fonction d'initialisation genre:

void main() {
    final List<YTheme> themes = [systemTheme, lightTheme, darkTheme];
    components.init(themes)
}

Ça voudrait dire qu'on ne peut pas utiliser d'énumération (YAvailableTheme).

En sachant qu'il faut pouvoir accéder à theme depuis le package ynotes_packages/theme.dart et non pas l'application.

T'en penses quoi @Androz2091 ?

Fix typos

The theme README on the rework branch contains several typos that must be corrected.

Rework: YForm: add parameter autosubmit

Autosubmit (bool): should submit the form automatically when pressing Done on the last input's keyboard. That means on onSubmit will never be triggered.
Concerning its usage, it can be to make sure the user clicks on a button with a clear label. Other use case: on submit, disable the form while processing data (for example login).

Add YThemeSwitcher dialog

Use the dialog currently used in the example.

Rework: add linter to example

Replace YButtonVariant by YVariant

Use ChangeNotifier to update the widget tree

Currently to refresh the widgets when a theme change, we use an empty setState in a tasteful widget. But this solution is more like a hack.
To fix that, the current theme class should extend ChangeNotifier to notify listeners when theme is updated.
So using the Provider package, we should wrap the MaterialApp with a Consumer. It must be added to YApp and also explained in the documentation for usage without that widget.

Rework the texts

We must what are all the text styles needed.
Concerning the link, it was supposed to be used with YButton but there are 2 solutions really:

• add a YButtonVariant
• add a new component (I prefer this approach but we must see)