Neues Popup Design #152

Merged
flixcoo merged 15 commits from feature/129-neues-popup-design into development 2026-01-13 22:01:05 +00:00
Owner

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.

# 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.
sneeex added this to the Alpha milestone 2026-01-11 10:49:09 +00:00
sneeex self-assigned this 2026-01-11 10:49:09 +00:00
sneeex requested review from Owners 2026-01-11 10:49:09 +00:00
sneeex added 1 commit 2026-01-11 14:26:14 +00:00
implement custom alert dialog
Some checks failed
Pull Request Pipeline / lint (pull_request) Failing after 2m34s
Pull Request Pipeline / test (pull_request) Successful in 2m33s
76ce3af643
Author
Owner

@flixcoo was sagst du dazu?

@flixcoo was sagst du dazu?
sneeex added 1 commit 2026-01-11 14:26:53 +00:00
add const
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m30s
Pull Request Pipeline / lint (pull_request) Successful in 2m31s
3ceae8341b
Owner

@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

> @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
Author
Owner

@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?

> > @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?
Owner

@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

> > > @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
sneeex added 1 commit 2026-01-11 16:14:31 +00:00
change button alignment & remove InkWell Animation
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m8s
Pull Request Pipeline / lint (pull_request) Successful in 2m10s
22ce742d43
sneeex added 1 commit 2026-01-11 16:21:08 +00:00
implement animation
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m4s
Pull Request Pipeline / lint (pull_request) Successful in 2m8s
1ebcfc9e57
Author
Owner

@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?

> > > > @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?
Owner

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?

> 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?
Author
Owner

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?

> > 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?
Owner

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

> 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
Author
Owner

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??

> > 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??
Author
Owner

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

in die richtung?
grafik.png

> > 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 in die richtung? <img width="232" alt="grafik.png" src="attachments/6fdc889a-fd59-4649-b7af-03dcc8caa7c5">
Author
Owner
grafik.png
<img width="240" alt="grafik.png" src="attachments/610429c7-c151-4acd-bc0a-1a9b6acb7b49">
Author
Owner
grafik.png
<img width="241" alt="grafik.png" src="attachments/dd8da593-7f44-47fb-8525-5ec04624cc44">
Author
Owner
grafik.png
<img width="243" alt="grafik.png" src="attachments/67a39cb6-9208-4c12-8847-ba6fd8c96490">
Owner
grafik.png

Ich glaube ich finde das am besten

> <img width="241" alt="grafik.png" src="attachments/dd8da593-7f44-47fb-8525-5ec04624cc44"> > Ich glaube ich finde das am besten
Owner

Oder was sagst du @gelbeinhalb

Oder was sagst du @gelbeinhalb
sneeex added 1 commit 2026-01-12 16:42:06 +00:00
Merge branch 'development' into feature/129-neues-popup-design
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m0s
Pull Request Pipeline / lint (pull_request) Successful in 2m12s
ab06662397
gelbeinhalb approved these changes 2026-01-12 20:32:04 +00:00
Dismissed
Owner

ups wollte nicht genehmigen xD

ups wollte nicht genehmigen xD
Owner
grafik.png

finde glaube ich untereinander am besten

> <img width="240" alt="grafik.png" src="attachments/610429c7-c151-4acd-bc0a-1a9b6acb7b49"> > finde glaube ich untereinander am besten
flixcoo requested review from gelbeinhalb 2026-01-13 12:57:34 +00:00
flixcoo requested review from flixcoo 2026-01-13 12:57:38 +00:00
flixcoo removed review request for gelbeinhalb 2026-01-13 12:57:40 +00:00
flixcoo requested review from gelbeinhalb 2026-01-13 12:57:41 +00:00
flixcoo declined to review 2026-01-13 12:57:42 +00:00
flixcoo removed review request for gelbeinhalb 2026-01-13 12:57:43 +00:00
flixcoo dismissed gelbeinhalb's review 2026-01-13 12:58:02 +00:00
Reason:

Noch WIP

Author
Owner

finde auch nebeneinander am besten, oben unten nimmt so viel unnötigen platz weg

finde auch nebeneinander am besten, oben unten nimmt so viel unnötigen platz weg
Author
Owner

also einfach nebeneinander?

also einfach nebeneinander?
Owner

also einfach nebeneinander?

Ja passt

> also einfach nebeneinander? Ja passt
sneeex added 2 commits 2026-01-13 19:45:52 +00:00
Merge remote-tracking branch 'origin/development' into feature/129-neues-popup-design
Some checks failed
Pull Request Pipeline / test (pull_request) Successful in 2m16s
Pull Request Pipeline / lint (pull_request) Failing after 2m38s
b69d2784df
sneeex added 1 commit 2026-01-13 19:46:45 +00:00
fix dart analysis errors
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m9s
Pull Request Pipeline / lint (pull_request) Successful in 2m16s
d662680a34
sneeex added 1 commit 2026-01-13 19:50:56 +00:00
add docs to custom_alert_dialog.dart
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m5s
Pull Request Pipeline / lint (pull_request) Successful in 2m10s
4161e1e88b
sneeex requested review from Owners 2026-01-13 19:51:43 +00:00
sneeex added the
PR
Ready for Review
2
label 2026-01-13 19:51:46 +00:00
sneeex changed title from WIP: Neues Popup Design to Neues Popup Design 2026-01-13 19:51:51 +00:00
flixcoo added
PR
In Review
1
and removed
PR
Ready for Review
2
labels 2026-01-13 19:55:17 +00:00
Author
Owner

macher, das geht aber schnell
grafik.png

macher, das geht aber schnell ![grafik.png](/attachments/0f14be88-6780-4a7a-99a1-09c1bfdb37ea)
Owner

macher, das geht aber schnell

😎

> macher, das geht aber schnell 😎
sneeex scheduled this pull request to auto merge when all checks succeed 2026-01-13 20:05:02 +00:00
flixcoo requested changes 2026-01-13 20:05:43 +00:00
Dismissed
flixcoo left a comment
Owner

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

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,
Owner

Punkt dahinter, also

content: '${loc.this_cannot_be_undone}.', 

(Oder zum String hinzufügen)

Punkt dahinter, also ```dart content: '${loc.this_cannot_be_undone}.', ``` (Oder zum String hinzufügen)
sneeex marked this conversation as resolved
@@ -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});
Owner

Konstrukturo über den Variablen

Konstrukturo über den Variablen
sneeex marked this conversation as resolved
@@ -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 {
Owner

Ähnlich wie bei allen Widget Klassen die Attribut-Variablen erklären

Ähnlich wie bei allen Widget Klassen die Attribut-Variablen erklären
flixcoo marked this conversation as resolved
@@ -0,0 +16,4 @@
/// These actions are horizontally spaced around the dialog's width.
final List<Widget> actions;
const CustomAlertDialog({
Owner

Konstruktor über den Variablen

Konstruktor über den Variablen
sneeex marked this conversation as resolved
Author
Owner

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

grafik.png ??
> 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 <img width="236" alt="grafik.png" src="attachments/ac18875f-f644-45cb-aa41-31ee8c041870"> ??
Author
Owner

find das aber eigentlich gut so
grafik.png

find das aber eigentlich gut so <img width="235" alt="grafik.png" src="attachments/2128d767-d2e5-4236-9288-6576d7c4ef32">
sneeex added 1 commit 2026-01-13 20:17:03 +00:00
refactor: enhance documentation and fix punctuation in localization strings
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m3s
Pull Request Pipeline / lint (pull_request) Successful in 2m9s
82ad2b74f8
flixcoo canceled auto merging this pull request when all checks succeed 2026-01-13 20:27:29 +00:00
Owner

find das aber eigentlich gut so
grafik.png

Was wäre wenn du den Primary Button den hintergrund lila machst, nicht den text

> find das aber eigentlich gut so > <img width="235" alt="grafik.png" src="attachments/2128d767-d2e5-4236-9288-6576d7c4ef32"> Was wäre wenn du den Primary Button den hintergrund lila machst, nicht den text
Author
Owner

find das aber eigentlich gut so
grafik.png

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?

> > find das aber eigentlich gut so > > <img width="235" alt="grafik.png" src="attachments/2128d767-d2e5-4236-9288-6576d7c4ef32"> > > 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?
Author
Owner

obwohl gibt beides glaube ich

obwohl gibt beides glaube ich
Author
Owner
grafik.png grafik.png
<img width="198" alt="grafik.png" src="attachments/f61aca77-b3b5-4fa9-954b-c8307a09ca53"> <img width="200" alt="grafik.png" src="attachments/f845cb65-54fc-4372-9597-e9766fb56809">
sneeex added 1 commit 2026-01-13 20:53:50 +00:00
add background color option to AnimatedDialogButton
Some checks failed
Pull Request Pipeline / lint (pull_request) Failing after 2m47s
Pull Request Pipeline / test (pull_request) Successful in 2m38s
4019ed083f
Owner
grafik.png grafik.png

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

> <img width="198" alt="grafik.png" src="attachments/f61aca77-b3b5-4fa9-954b-c8307a09ca53"> > <img width="200" alt="grafik.png" src="attachments/f845cb65-54fc-4372-9597-e9766fb56809"> 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
sneeex added 1 commit 2026-01-13 21:17:51 +00:00
Revert "add background color option to AnimatedDialogButton"
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m5s
Pull Request Pipeline / lint (pull_request) Successful in 2m19s
db51990695
This reverts commit 4019ed083f.
sneeex requested review from flixcoo 2026-01-13 21:18:55 +00:00
flixcoo requested changes 2026-01-13 21:49:43 +00:00
Dismissed
@@ -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});
Owner

Docstring über den Konstruktor, nicht über die Klasse (siehe #167)

Docstring über den Konstruktor, nicht über die Klasse (siehe #167)
flixcoo marked this conversation as resolved
@@ -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.
Owner

newline hinter onPressed

newline hinter `onPressed`
flixcoo marked this conversation as resolved
Author
Owner

Kannst du das eben machen und dann mergen?

Kannst du das eben machen und dann mergen?
Author
Owner

Bin schon im Bett

Bin schon im Bett
Owner

Kannst du das eben machen und dann mergen?
Bin schon im Bett

ja okay

> Kannst du das eben machen und dann mergen? > Bin schon im Bett ja okay
flixcoo added 3 commits 2026-01-13 21:58:39 +00:00
# Conflicts:
#	lib/presentation/views/main_menu/settings_view/settings_view.dart
Merge cleaning
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m2s
Pull Request Pipeline / lint (pull_request) Successful in 2m7s
2cadab004d
flixcoo requested review from flixcoo 2026-01-13 21:59:34 +00:00
flixcoo approved these changes 2026-01-13 21:59:38 +00:00
flixcoo scheduled this pull request to auto merge when all checks succeed 2026-01-13 22:01:03 +00:00
flixcoo merged commit bbd41a65df into development 2026-01-13 22:01:05 +00:00
flixcoo deleted branch feature/129-neues-popup-design 2026-01-13 22:01:05 +00:00
Sign in to join this conversation.
No Reviewers
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: liquid-development/game-tracker#152