Task/185 component toasts #12
Conversation
Costem Tost gemäß den Angaben im Lastenheft. -Textinhalt kann vorgegeben werden -Einstellbarer Schweregrad “success”, “warning”, “danger” -Farben/Form gemäß Lastenheft -Integration der Abhängigkeit -Anpassung minSdkVersion
-Anpassung minSdkVersion
|
Bestenfalls sollte die Funktion so eingebunden sein. WidgetsBinding.instance.addPostFrameCallback((_){
showToast(context, "Hallo Alex", "danger");
}); |
|
Das Aussehen ist gut, leider sind noch ein paar Dinge Schreibfehler: massage statt message Der Status kann auch in ein Enum gepackt werden Warum muss die Sdk Version geändert werden? |
This reverts commit d765e8d1b0a85ce3de9f1634a47540ddd0dc0f2e.
lib/components/Toast.dart
Outdated
| /// example call: | ||
| /// showToast(context,"Hello Wold","success"); | ||
| /// | ||
| showToast(BuildContext context,String massage, String level) { |
There was a problem hiding this comment.
Typo, message statt massage
lib/components/Toast.dart
Outdated
| final Color textColor; | ||
| final IconData icon; | ||
|
|
||
| if (level == "success") { |
There was a problem hiding this comment.
switch statement wäre hier geeigneter
lib/components/Toast.dart
Outdated
| /// example call: | ||
| /// showToast(context,"Hello Wold","success"); | ||
| /// | ||
| showToast(BuildContext context,String massage, String level) { |
There was a problem hiding this comment.
level gerne gegen ein enum austauschen
android/app/build.gradle
Outdated
| @@ -45,7 +45,7 @@ android { | |||
| applicationId "com.example.biersommelier" | |||
| // You can update the following values to match your application needs. | |||
| // For more information, see: https://docs.flutter.dev/deployment/android#reviewing-the-gradle-build-configuration. | |||
| minSdkVersion 20 | |||
| minSdkVersion 21 | |||
There was a problem hiding this comment.
wofür brauchen wir die neuere Version?
-minSdkVersion reset -level as enum -spelling error adjusted
|
Ich habe die Punkte jetzt angepasst. -Enum kannte ich noch nicht. Super Sache! Bin mir nicht sicher ob mein Comit sich jetzt direkt hier wiederspiegelt oder ich etwas in github klickeb muss. |
|
Super, das Switch sickert leider nun durch alle states durch. Ich habe einen branch bei dir im Fork erstellt welcher alle Probleme fixen sollte (Switch bug, Formatierung, Typo, Sdk revert, in |
Code OP#185
Funktion showToast muss mit Parameter
aufgerufen werden.