liquid-development/game-tracker
3
1
Fork 0
Code Issues 9 Pull Requests 3 Actions Releases 1 Activity

Popup-Buttons überarbeiten #199

Merged
flixcoo merged 6 commits from enhancement/popup-buttons-ueberarbeiten into development 2026-04-21 08:46:46 +00:00
Conversation 0 Commits 6 Files Changed 6 +122 -70
flixcoo commented 2026-04-19 14:07:29 +00:00
Owner
Copy Link

Zugehörige Issue(s)

Closes #198

Beschreibung

Es wurden der Stil der Buttons der Popups überarbeitet. Außerdem wurde die Struktur leicht überarbeitet so dass der AnimatedDialogButton jetzt etwas mehr abgekapselt ist vom eigentlichen Dialog und dadurch ggf. in Zukunft auch einzelnd verwendet werden kann.

Änderungen

  • Neues Widget CustomDialogAction hinzugefügt, welches den AnimatedDialogButton kapselt
  • Styling vom AnimatedDialogButton abhängig vom übergebenen ButtonType gemacht
  • Attribute vom CustomAlertDialog überarbeitet
    • content ist jetzt ein Widget, kann also auch mehr als nur einen Text darstellen
    • actions wurde von List<Widget> zu List<CustomDialogAction> geändert
  • Popups in den Views an neue Struktur angepasst

Zusätzliche Anmerkungen

No response

### Zugehörige Issue(s) Closes #198 ### Beschreibung Es wurden der Stil der Buttons der Popups überarbeitet. Außerdem wurde die Struktur leicht überarbeitet so dass der `AnimatedDialogButton` jetzt etwas mehr abgekapselt ist vom eigentlichen Dialog und dadurch ggf. in Zukunft auch einzelnd verwendet werden kann. ### Änderungen - [x] Neues Widget `CustomDialogAction` hinzugefügt, welches den `AnimatedDialogButton` kapselt - [x] Styling vom `AnimatedDialogButton` abhängig vom übergebenen `ButtonType` gemacht - [x] Attribute vom `CustomAlertDialog` überarbeitet - `content` ist jetzt ein Widget, kann also auch mehr als nur einen Text darstellen - `actions` wurde von `List<Widget>` zu `List<CustomDialogAction>` geändert - [x] Popups in den Views an neue Struktur angepasst ### Zusätzliche Anmerkungen _No response_
flixcoo added this to the Alpha milestone 2026-04-19 14:07:29 +00:00
flixcoo self-assigned this 2026-04-19 14:07:29 +00:00
flixcoo added 6 commits 2026-04-19 14:07:29 +00:00
Add: custom dialog action 15702a108d
Changed popup class structure 55437d83c4
Updated popups in views 32e1c587d4
Updated documentation 2ad3698067
Added constraint parameter ad2e4bc398
Moved button
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 46s
Details
Pull Request Pipeline / lint (pull_request) Successful in 48s
Details
9c4eff5056
flixcoo requested review from Owners 2026-04-19 14:07:40 +00:00
flixcoo added the
PR
Ready for Review
2
 label 2026-04-19 14:07:44 +00:00
flixcoo changed title from WIP: Popup-Buttons überarbeiten to Popup-Buttons überarbeiten 2026-04-19 14:07:48 +00:00
gelbeinhalb approved these changes 2026-04-21 08:39:24 +00:00
lib/presentation/widgets/buttons/animated_dialog_button.dart
@@ -30,0 +37,4 @@
color: widget.buttonType == ButtonType.primary
? Colors.black
: Colors.white,
fontSize: 16,
gelbeinhalb commented 2026-04-21 08:38:29 +00:00
Owner
Copy Link

können wir sowas hier eigentlich mal zentraler aufschreiben? Sonst sind das so magic numbers im code

können wir sowas hier eigentlich mal zentraler aufschreiben? Sonst sind das so magic numbers im code
lib/presentation/widgets/buttons/animated_dialog_button.dart
@@ -30,0 +45,4 @@
// Primary
? BoxDecoration(
color: Colors.white,
borderRadius: BorderRadius.circular(12),
gelbeinhalb commented 2026-04-21 08:38:39 +00:00
Owner
Copy Link

sowas hier halt auch

sowas hier halt auch
lib/presentation/widgets/buttons/animated_dialog_button.dart
@@ -45,0 +71,4 @@
child: Container(
constraints: widget.buttonConstraints,
decoration: buttonDecoration,
padding: const EdgeInsets.symmetric(horizontal: 16, vertical: 12),
gelbeinhalb commented 2026-04-21 08:39:01 +00:00
Owner
Copy Link

keine ahnung ob ich das jetzt überall markieren sollte, aber ich denke ihr checkt was ich meine

keine ahnung ob ich das jetzt überall markieren sollte, aber ich denke ihr checkt was ich meine
lib/presentation/widgets/buttons/animated_dialog_button.dart
@@ -45,0 +72,4 @@
constraints: widget.buttonConstraints,
decoration: buttonDecoration,
padding: const EdgeInsets.symmetric(horizontal: 16, vertical: 12),
margin: const EdgeInsets.symmetric(vertical: 8),
gelbeinhalb commented 2026-04-21 08:39:21 +00:00
Owner
Copy Link

hat aber auch nichts mit diesem pr direkt zu tun

hat aber auch nichts mit diesem pr direkt zu tun
flixcoo merged commit c63f9067d3 into development 2026-04-21 08:46:46 +00:00
flixcoo deleted branch enhancement/popup-buttons-ueberarbeiten 2026-04-21 08:46:47 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
gelbeinhalb
Labels
Clear labels
Duplicate
This issue or pull request already exists
Layer
BE
Issues/PRs related to the backend
Layer
DB
Issues/PRs related to the database
Layer
FE
Issues/PRs related to the frontend
Layer
Full Stack
Issues/PRs spanning multiple layers (frontend, backend, database)
PR
In Review
1
Pull Request that is reviewed
PR
Ready for Review
2
Pull Requests which are ready to be reviewed
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Low
4
The priority is low
Priority
Medium
3
The priority is medium
Proposal
This issue contains a proposal for a new feature, change, or improvement. It is open for discussion and feedback before any decision is made.
Task
Bug
Reports of unexpected behavior or defects in existing functionality.
Task
Documentation
Tasks related to improving or adding documentation (READMEs, code comments, guides).
Task
Enhancement
Improvements or optimizations to existing features (non-breaking changes).
Task
Feature
Requests for new functionality or significant additions to the project.
Task
Hotfix
Task
Refactoring
Tasks that only involve refactoring and do not add any new functionality
Task
Setup
Tasks related to project configuration, tooling, or infrastructure.
Task
Testing
Issue or pull request related to testing
Technical Debt
Issues related to code that needs to be refactored, cleaned up, or improved due to shortcuts or outdated implementations and to keep the codebase maintainable.
Workflow
Abandoned
4
Somebody has started to work on this but abandoned work
Workflow
Backlog
4
Issues/PRs that are acknowledged but not being actively worked on yet.
Workflow
In Progress
3
Issues/PRs currently being actively worked on.
Workflow
On Hold
3
Issues/PRs temporarily being paused or blocked.
Workflow
Won't Fix
5
This issue won't be fixed
No Label
PR
Ready for Review
2
Milestone
No items
No Milestone Alpha
Assignees
Clear assignees
actions flixcoo gelbeinhalb sneeex
No Assignees flixcoo
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: liquid-development/game-tracker#199
Delete Branch "enhancement/popup-buttons-ueberarbeiten"

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?