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

TopCenteredMessage Anzeigefehler #161

Merged
sneeex merged 2 commits from bug/159-topcenteredmessage-anzeigefehler into development 2026-01-13 14:30:53 +00:00
Conversation 10 Commits 2 Files Changed 3 +9 -2
flixcoo commented 2026-01-12 22:21:51 +00:00
Owner
Copy Link

Zugehörige Issue(s)

Closes #159

Beschreibung

Behebt den Pixel Overflow der TopCenteredMessage in der PlayerSelection, wenn diese relativ klein war (z.B. CreateMatchView)

Änderungen

  • fullscreen Attribut zu TopCenteredMessage hinzugefügt, bei welchem das Top Padding entfernt und das Alignment auf mainAxisAlignment.center gesetzt wird

Zusätzliche Anmerkungen

No response

### Zugehörige Issue(s) Closes #159 ### Beschreibung Behebt den Pixel Overflow der `TopCenteredMessage` in der `PlayerSelection`, wenn diese relativ klein war (z.B. `CreateMatchView`) ### Änderungen - `fullscreen` Attribut zu `TopCenteredMessage` hinzugefügt, bei welchem das Top Padding entfernt und das Alignment auf `mainAxisAlignment.center` gesetzt wird ### Zusätzliche Anmerkungen _No response_
flixcoo added 1 commit 2026-01-12 22:21:51 +00:00
Fixed displayment bug with TopCenteredMessage
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m6s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m11s
Details
d5bd0bca5f
flixcoo requested review from Owners 2026-01-12 22:21:57 +00:00
flixcoo self-assigned this 2026-01-12 22:21:59 +00:00
flixcoo added this to the Alpha milestone 2026-01-12 22:22:02 +00:00
flixcoo added the
PR
Ready for Review
2
 label 2026-01-12 22:22:04 +00:00
sneeex commented 2026-01-13 14:09:11 +00:00
Owner
Copy Link

du meinst overflow wenn man tastatur rausholt?

du meinst overflow wenn man tastatur rausholt?
flixcoo commented 2026-01-13 14:10:54 +00:00
Author
Owner
Copy Link

du meinst overflow wenn man tastatur rausholt?

Nein, das ist schon passiert wenn man garnicht die Tastatur draußen hatte in der CreateGameView weil da die Player Selection so klein ist

> du meinst overflow wenn man tastatur rausholt? Nein, das ist schon passiert wenn man garnicht die Tastatur draußen hatte in der `CreateGameView` weil da die Player Selection so klein ist
sneeex commented 2026-01-13 14:13:45 +00:00
Owner
Copy Link

du meinst overflow wenn man tastatur rausholt?

Nein, das ist schon passiert wenn man garnicht die Tastatur draußen hatte in der CreateGameView weil da die Player Selection so klein ist

mach mal video, checke nicht was du meinst

> > du meinst overflow wenn man tastatur rausholt? > > Nein, das ist schon passiert wenn man garnicht die Tastatur draußen hatte in der `CreateGameView` weil da die Player Selection so klein ist mach mal video, checke nicht was du meinst
sneeex commented 2026-01-13 14:14:41 +00:00
Owner
Copy Link

Ich kann das nicht reproduzieren, siehe Anhang. Nur mit Tastatur

Ich kann das nicht reproduzieren, siehe Anhang. Nur mit Tastatur
Bildschirmaufnahme 2026-01-13 um 15.13.15.mov
1.4 MiB
flixcoo commented 2026-01-13 14:17:14 +00:00
Author
Owner
Copy Link

mach mal video, checke nicht was du meinst

> mach mal video, checke nicht was du meinst
Simulator Screen Recording - iPhone 17 - 2026-01-13 at 15.16.40.mov
433 KiB
sneeex commented 2026-01-13 14:22:21 +00:00
Owner
Copy Link

ist nen ios ding glaube ich, bei mir gehts ja
grafik.png

grafik.png
ist nen ios ding glaube ich, bei mir gehts ja <img width="272" alt="grafik.png" src="attachments/0e3be85b-10e2-4964-a860-1b7013159575"> <img width="292" alt="grafik.png" src="attachments/013e7bf4-e368-486e-83ed-ac3dc40dabf7">
grafik.png
81 KiB
grafik.png
85 KiB
flixcoo commented 2026-01-13 14:23:10 +00:00
Author
Owner
Copy Link

ist nen ios ding glaube ich, bei mir gehts ja
grafik.png

grafik.png

Ja gut, aber sollte ja deswegen trotzdem gefixt werden

> ist nen ios ding glaube ich, bei mir gehts ja > <img width="272" alt="grafik.png" src="attachments/0e3be85b-10e2-4964-a860-1b7013159575"> > > <img width="292" alt="grafik.png" src="attachments/013e7bf4-e368-486e-83ed-ac3dc40dabf7"> > Ja gut, aber sollte ja deswegen trotzdem gefixt werden
sneeex commented 2026-01-13 14:23:13 +00:00
Owner
Copy Link

warum ist das bei dir so groß alles?

warum ist das bei dir so groß alles?
flixcoo commented 2026-01-13 14:25:08 +00:00
Author
Owner
Copy Link

warum ist das bei dir so groß alles?

Keine Ahnung, ist hal so

> warum ist das bei dir so groß alles? Keine Ahnung, ist hal so
flixcoo added 1 commit 2026-01-13 14:26:05 +00:00
Merge branch 'development' into bug/159-topcenteredmessage-anzeigefehler
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m3s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m12s
Details
57357f8aad 
# Conflicts:
#	pubspec.yaml
sneeex commented 2026-01-13 14:30:21 +00:00
Owner
Copy Link

dann vertrau ich dir mal

dann vertrau ich dir mal
sneeex approved these changes 2026-01-13 14:30:47 +00:00
sneeex merged commit 7a1752f773 into development 2026-01-13 14:30:53 +00:00
sneeex deleted branch bug/159-topcenteredmessage-anzeigefehler 2026-01-13 14:30:53 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
sneeex
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
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
Task
Setup
Tasks related to project configuration, tooling, or infrastructure.
Task
Testing
Issue or pull request related to testing
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
flixcoo gelbeinhalb sneeex
No Assignees flixcoo
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: liquid-development/game-tracker#161
Delete Branch "bug/159-topcenteredmessage-anzeigefehler"

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?