Neues Popup Design #152
Reference in New Issue
Block a user
Delete Branch "feature/129-neues-popup-design"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Neues Popup Design
Zugehörige Issue(s):
Closes #129
Beschreibung
Ein neues Popup Design implementiert, was das Design der App nutzt und nicht OS Spezifisch aussieht.
@flixcoo was sagst du dazu?
Ja sieht nicht schlecht aus, würde die Buttons aber ohne den InkWell machen, das sieht zu Android like aus bzw. vllt hast du noch ne bessere Idee für Buttons. Maybe auch Buttons nicht neben sondern untereinander? So n bisschen ios popup like
hä aber gibts auf ios kein visuelles feedback beim button?
Doch eben schon, aber das sieht zu Android-Like aus, das ist dieses typische InkWell
so?
Ja, find die Animation auf jeden fall so gut. Könntest du trotzdem mal eine Variante entweder mit Buttons die mehr als Text sind (also richtiger Button) oder mal die buttons vertikal angeordnet versuchen?
Weiß nicht wie das vertikal geht, ohne dass ich nen eigenen Alertdialog implementieren, du?
Ist glaub ich nicht so schwer, siehe: https://stackoverflow.com/questions/53019061/how-to-implement-a-custom-dialog-box-in-flutter
das ist doch nen custom dialog??
in die richtung?

Ich glaube ich finde das am besten
Oder was sagst du @gelbeinhalb
ups wollte nicht genehmigen xD
finde glaube ich untereinander am besten
Noch WIP
finde auch nebeneinander am besten, oben unten nimmt so viel unnötigen platz weg
also einfach nebeneinander?
Ja passt
WIP: Neues Popup Designto Neues Popup Designmacher, das geht aber schnell

😎
Irgendwie bin ich mir noch nicht ganz sicher beim Button Design, ich find das irgendwie noch nicht so perfekt. Kannst du da vllt noch was anderes versuchen, maybe den button hintergrund nicht den gleichen wie den popup hintergrund
@@ -104,1 +104,3 @@content: Text(loc.this_cannot_be_undone),builder: (context) => CustomAlertDialog(title: '${loc.delete_all_data}?',content: loc.this_cannot_be_undone,Punkt dahinter, also
(Oder zum String hinzufügen)
@@ -0,0 +14,4 @@/// Creates an instance of `AnimatedDialogButton`.////// The [onPressed] and [child] parameters are required.const AnimatedDialogButton({super.key, required this.onPressed, required this.child});Konstrukturo über den Variablen
@@ -0,0 +5,4 @@////// This widget provides a styled alternative to the default Flutter AlertDialog,/// with consistent colors, borders, and layout that match the app's custom theme.class CustomAlertDialog extends StatelessWidget {Ähnlich wie bei allen Widget Klassen die Attribut-Variablen erklären
@@ -0,0 +16,4 @@/// These actions are horizontally spaced around the dialog's width.final List<Widget> actions;const CustomAlertDialog({Konstruktor über den Variablen
find das aber eigentlich gut so

Was wäre wenn du den Primary Button den hintergrund lila machst, nicht den text
sollte nicht eher der secondary wenn dann lila sein?
wegen so leute vom löschen abbringen?
obwohl gibt beides glaube ich
Ja obwohl bin auch nicht so sicher, sonst mach erstmal was du denkst, und dann mergen wir erstmal. Können ja immer nochmal drüber gehen
@@ -0,0 +9,4 @@/// - [onPressed]: Callback function that is triggered when the button is pressed./// - [child]: The child widget to be displayed inside the button, typically a text or icon.class AnimatedDialogButton extends StatefulWidget {const AnimatedDialogButton({super.key, required this.onPressed, required this.child});Docstring über den Konstruktor, nicht über die Klasse (siehe #167)
@@ -0,0 +13,4 @@/// Callback function that is triggered when the button is pressed.final VoidCallback onPressed;/// The child widget to be displayed inside the button, typically a text or icon.newline hinter
onPressedKannst du das eben machen und dann mergen?
Bin schon im Bett
ja okay