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

Haptisches Feedback hinzufügen #216

Merged
flixcoo merged 14 commits from feature/215-haptisches-feedback-hinzufügen into development 2026-05-14 13:09:01 +00:00
Conversation 12 Commits 14 Files Changed 26 +328 -68
sneeex commented 2026-05-10 21:05:18 +00:00
Owner
Copy Link

Zugehörige Issue(s)

Closes #215

Beschreibung

Haptisches Feedback hinzugefügt

Änderungen

  • Bei den meisten Interaktionen selectionClick() hinzugefügt, bei anderen (z.B. Long Press) vibrate() und bei anderen mit Success/Failure .successNotification() bzw. .errorNotification()
  • Drei neue Widgets jeweils als Standard gesetzt, die Vibration eingebaut haben (haptic_icon_button, haptic_close_button, haptic_back_button)
### Zugehörige Issue(s) Closes #215 ### Beschreibung Haptisches Feedback hinzugefügt ### Änderungen - Bei den meisten Interaktionen `selectionClick()` hinzugefügt, bei anderen (z.B. Long Press) `vibrate()` und bei anderen mit Success/Failure `.successNotification()` bzw. `.errorNotification()` - Drei neue Widgets jeweils als Standard gesetzt, die Vibration eingebaut haben (`haptic_icon_button`, `haptic_close_button`, `haptic_back_button`)
sneeex added this to the Alpha milestone 2026-05-10 21:05:18 +00:00
sneeex self-assigned this 2026-05-10 21:05:18 +00:00
sneeex added 1 commit 2026-05-10 21:05:18 +00:00
feat: add haptic feedback for various user interactions
Some checks failed
Pull Request Pipeline / test (pull_request) Successful in 45s
Details
Pull Request Pipeline / lint (pull_request) Failing after 48s
Details
1d20127af4
sneeex added 1 commit 2026-05-11 08:33:16 +00:00
feat: add haptic feedback to more user interactions
Some checks failed
Pull Request Pipeline / test (pull_request) Successful in 49s
Details
Pull Request Pipeline / lint (pull_request) Failing after 50s
Details
bc59d1d91c
sneeex added 1 commit 2026-05-11 08:59:51 +00:00
feat: add haptic feedback to even more user interactions
Some checks failed
Pull Request Pipeline / test (pull_request) Successful in 48s
Details
Pull Request Pipeline / lint (pull_request) Failing after 51s
Details
f1899bfe44
sneeex commented 2026-05-11 09:00:38 +00:00
Author
Owner
Copy Link

@flixcoo guck mal ob das auf ios geht
wenn nicht dann geh ich sterben

@flixcoo guck mal ob das auf ios geht wenn nicht dann geh ich sterben
sneeex commented 2026-05-11 18:46:08 +00:00
Author
Owner
Copy Link
grafik.png
<img width="525" alt="grafik.png" src="attachments/c72c703b-9d80-4ad3-89a2-982525c0f981">
grafik.png
41 KiB
sneeex added 1 commit 2026-05-11 19:19:57 +00:00
Merge branch 'development' into feature/215-haptisches-feedback-hinzufügen
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 46s
Details
Pull Request Pipeline / lint (pull_request) Successful in 55s
Details
1edad99a1c
sneeex changed title from WIP: Haptisches Feedback hinzufügen to Haptisches Feedback hinzufügen 2026-05-11 19:20:06 +00:00
flixcoo changed title from Haptisches Feedback hinzufügen to WIP: Haptisches Feedback hinzufügen 2026-05-11 19:22:54 +00:00
sneeex changed title from WIP: Haptisches Feedback hinzufügen to Haptisches Feedback hinzufügen 2026-05-11 19:25:27 +00:00
sneeex changed title from Haptisches Feedback hinzufügen to WIP: Haptisches Feedback hinzufügen 2026-05-11 19:25:29 +00:00
sneeex requested review from Owners 2026-05-11 19:25:32 +00:00
sneeex changed title from WIP: Haptisches Feedback hinzufügen to Haptisches Feedback hinzufügen 2026-05-11 19:25:33 +00:00
sneeex added the
PR
Ready for Review
2
 label 2026-05-11 19:26:05 +00:00
flixcoo requested changes 2026-05-12 20:20:11 +00:00
Dismissed
flixcoo left a comment
Owner
Copy Link

Generell gefällt mir das sehr gut, gibt der App irgendwie ein lebendigeres Gefühl. Paar Kleinigkeiten sind noch dabei.

Generell gefällt mir das sehr gut, gibt der App irgendwie ein lebendigeres Gefühl. Paar Kleinigkeiten sind noch dabei.
lib/core/custom_theme.dart
@@ -83,6 +85,11 @@ class CustomTheme {
iconTheme: IconThemeData(color: textColor),
);
static final ActionIconThemeData actionIconTheme = ActionIconThemeData(
flixcoo commented 2026-05-12 20:13:48 +00:00
Owner
Copy Link

Bitte überprüfen: So wie ich getestet haben kann man an diversen Stellen die Buttons weglassen, wenn sie nur navigation übernehmen. Bitte alle Buttons in den App Bars entfernen, wenn das tatsächlich möglich ist.

Bitte überprüfen: So wie ich getestet haben kann man an diversen Stellen die Buttons weglassen, wenn sie nur navigation übernehmen. Bitte alle Buttons in den App Bars entfernen, wenn das tatsächlich möglich ist.
sneeex commented 2026-05-13 10:24:42 +00:00
Author
Owner
Copy Link

hab einmal per search alles mit appbar angeguckt, die buttons sind nur da explizit drin, wo sie auch gebraucht werden, weil sie z.B. mit einer variable den screen poppen

hab einmal per search alles mit appbar angeguckt, die buttons sind nur da explizit drin, wo sie auch gebraucht werden, weil sie z.B. mit einer variable den screen poppen
flixcoo commented 2026-05-13 10:25:17 +00:00
Owner
Copy Link

Alles klar, dann passt

Alles klar, dann passt
sneeex commented 2026-05-13 10:35:14 +00:00
Author
Owner
Copy Link

aber grundsätzlich hast du recht, standardmäßig ist der zurück button immer da

aber grundsätzlich hast du recht, standardmäßig ist der zurück button immer da
flixcoo marked this conversation as resolved
lib/presentation/views/main_menu/custom_navigation_bar.dart
@@ -58,2 +60,2 @@
await Navigator.push(
context,
final navigator = Navigator.of(context);
await HapticFeedback.selectionClick();
flixcoo commented 2026-05-12 20:08:14 +00:00
Owner
Copy Link

Warum rufst du hier extra nochmal HapticFeedback.selectionClick() auf?

Warum rufst du hier extra nochmal `HapticFeedback.selectionClick()` auf?
sneeex commented 2026-05-13 10:26:16 +00:00
Author
Owner
Copy Link

war wohl ausversehen, vmtl noch aus dem testing, habs entfernt

war wohl ausversehen, vmtl noch aus dem testing, habs entfernt
sneeex marked this conversation as resolved
lib/presentation/views/main_menu/settings_view/settings_view.dart Outdated
@@ -201,7 +202,8 @@ class _SettingsViewState extends State<SettingsView> {
children: [
GestureDetector(
child: const Icon(Icons.language),
onTap: () => {
flixcoo commented 2026-05-12 20:18:02 +00:00
Owner
Copy Link

Kann man diese drei Icon Buttons nicht direkt durch HapticIconButton ersetzen?

Kann man diese drei Icon Buttons nicht direkt durch `HapticIconButton` ersetzen?
sneeex commented 2026-05-12 21:00:51 +00:00
Author
Owner
Copy Link

Kann man, aber der hat doch nen ganz anderes Design oder nicht?

Kann man, aber der hat doch nen ganz anderes Design oder nicht?
flixcoo commented 2026-05-13 10:22:25 +00:00
Owner
Copy Link

Hab’s mir jetzt nicht genau angeschaut aber ist doch eigentlich auch nur n clickable Icon.

Hab’s mir jetzt nicht genau angeschaut aber ist doch eigentlich auch nur n clickable Icon.
sneeex marked this conversation as resolved
lib/presentation/widgets/buttons/main_menu_button.dart Outdated
@@ -78,6 +79,7 @@ class _MainMenuButtonState extends State<MainMenuButton>
onTapUp: (_) async {
_cancelTimers();
if (mounted && !_isLongPressing) {
await HapticFeedback.selectionClick();
flixcoo commented 2026-05-12 20:19:21 +00:00
Owner
Copy Link

Warum ist hier nur auf dem Short Tab eine Vibration und nicht auf dem Long Tap?

Warum ist hier nur auf dem Short Tab eine Vibration und nicht auf dem Long Tap?
sneeex commented 2026-05-13 10:32:58 +00:00
Author
Owner
Copy Link

nicht gecheckt, habs geändert

nicht gecheckt, habs geändert
sneeex marked this conversation as resolved
flixcoo added
PR
In Review
1
 and removed
PR
Ready for Review
2
 labels 2026-05-12 20:20:18 +00:00
flixcoo added spent time 10 minutes 2026-05-12 20:20:22 +00:00
gelbeinhalb commented 2026-05-12 20:32:24 +00:00
Owner
Copy Link

kann ich garnicht testen 😓

kann ich garnicht testen 😓
sneeex commented 2026-05-12 21:02:33 +00:00
Author
Owner
Copy Link

kann ich garnicht testen 😓

Imagine man hat kein Dev Handy

> kann ich garnicht testen 😓 > Imagine man hat kein Dev Handy
sneeex closed this pull request 2026-05-12 21:02:33 +00:00
sneeex reopened this pull request 2026-05-12 21:02:38 +00:00
sneeex commented 2026-05-12 21:02:49 +00:00
Author
Owner
Copy Link

ups

ups
sneeex added 4 commits 2026-05-13 10:45:04 +00:00
remove unnecessary click b82cca939f
add vibration to long tap 7e35ccae6b
change settings_view buttons to haptic icon button a8369249b4
Merge remote-tracking branch 'origin/feature/215-haptisches-feedback-hinzufügen' into feature/215-haptisches-feedback-hinzufügen
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 48s
Details
Pull Request Pipeline / lint (pull_request) Successful in 54s
Details
02bb7e4daa
sneeex commented 2026-05-13 10:45:32 +00:00
Author
Owner
Copy Link

Generell gefällt mir das sehr gut, gibt der App irgendwie ein lebendigeres Gefühl. Paar Kleinigkeiten sind noch dabei.

finde aber auch

> Generell gefällt mir das sehr gut, gibt der App irgendwie ein lebendigeres Gefühl. Paar Kleinigkeiten sind noch dabei. finde aber auch
sneeex requested review from flixcoo 2026-05-13 10:45:38 +00:00
flixcoo approved these changes 2026-05-13 11:51:09 +00:00
Dismissed
lib/presentation/widgets/buttons/haptic_icon_button.dart
@@ -0,0 +1,54 @@
import 'package:flutter/material.dart';
import 'package:flutter/services.dart';
class HapticIconButton extends StatelessWidget {
flixcoo commented 2026-05-13 11:50:41 +00:00
Owner
Copy Link

Kannst du bitte noch bei den drei neuen Buttons den Splash-Effekt entfernen und eine Scale Animation hinzufügen, ähnlich wie bei main menu button? Ggf. ersetzt du dann IconButton() einfach durch einen Container(), vllt ist das sogar einfacher

Kannst du bitte noch bei den drei neuen Buttons den Splash-Effekt entfernen und eine Scale Animation hinzufügen, ähnlich wie bei main menu button? Ggf. ersetzt du dann `IconButton()` einfach durch einen `Container()`, vllt ist das sogar einfacher
sneeex commented 2026-05-13 13:36:39 +00:00
Author
Owner
Copy Link

du meinst bei allen IconButtons? weil das ist ja dann nicht nur die drei buttons in den settings, sondern alle back buttons auch. Und was für ein scale meinst du? icon scale? button scale insgesamt? und wieso? finde das eigentlich glaube nicht so geil

du meinst bei allen IconButtons? weil das ist ja dann nicht nur die drei buttons in den settings, sondern alle back buttons auch. Und was für ein scale meinst du? icon scale? button scale insgesamt? und wieso? finde das eigentlich glaube nicht so geil
flixcoo commented 2026-05-13 14:23:29 +00:00
Owner
Copy Link

Ja bei allen Icon Buttons.
Meinetwegen auch ohne Scale Effekt, aber auf jeden Fall ohne Splash. Und mit Scale meinte ich dass das Icon sich in der Größe verändert, wie beim MainMenuButton

Ja bei allen Icon Buttons. Meinetwegen auch ohne Scale Effekt, aber auf jeden Fall ohne Splash. Und mit Scale meinte ich dass das Icon sich in der Größe verändert, wie beim `MainMenuButton`
sneeex commented 2026-05-13 14:24:41 +00:00
Author
Owner
Copy Link

Ja bei allen Icon Buttons.
Meinetwegen auch ohne Scale Effekt, aber auf jeden Fall ohne Splash. Und mit Scale meinte ich dass das Icon sich in der Größe verändert, wie beim MainMenuButton

Beim Main Menu Button ist aber der ganze Button mit Scale und nicht nur das Icon

> Ja bei allen Icon Buttons. > Meinetwegen auch ohne Scale Effekt, aber auf jeden Fall ohne Splash. Und mit Scale meinte ich dass das Icon sich in der Größe verändert, wie beim `MainMenuButton` Beim Main Menu Button ist aber der ganze Button mit Scale und nicht nur das Icon
flixcoo commented 2026-05-13 14:25:26 +00:00
Owner
Copy Link

Der Icon Button ist ja aber nur n Icon, der hat ja keinen Hintergrund

Der Icon Button ist ja aber nur n Icon, der hat ja keinen Hintergrund
sneeex marked this conversation as resolved
lib/presentation/widgets/buttons/haptic_icon_button.dart
@@ -0,0 +33,4 @@
@override
Widget build(BuildContext context) {
return IconButton(
tooltip: tooltip,
flixcoo commented 2026-05-13 11:51:07 +00:00
Owner
Copy Link

Nicht sinnvoll, wir sind ja auf mobile, da gibts ja keinen hover state

Nicht sinnvoll, wir sind ja auf mobile, da gibts ja keinen hover state
sneeex marked this conversation as resolved
flixcoo requested changes 2026-05-13 11:51:40 +00:00
Dismissed
flixcoo left a comment
Owner
Copy Link

Ausversehen auf Gehnemigen gedrückt

Ausversehen auf Gehnemigen gedrückt
flixcoo added 1 commit 2026-05-13 11:52:47 +00:00
fix: long press feedback on every call
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 47s
Details
Pull Request Pipeline / lint (pull_request) Successful in 53s
Details
f5fbb3ecc4
flixcoo added 1 commit 2026-05-13 11:54:08 +00:00
changed long press vibration
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 48s
Details
Pull Request Pipeline / lint (pull_request) Successful in 54s
Details
e2882c1c6a
flixcoo commented 2026-05-13 11:54:47 +00:00
Owner
Copy Link

Hab bei zwei long presses das Feedback überarbeitet, beim Live Edit wollte ich das gerne jedes mal wenn long press gecalled wird und beim game tile hat sich die vibration falsch angefühlt. check das gerne nochmal ob das auf android auch passt

Hab bei zwei long presses das Feedback überarbeitet, beim Live Edit wollte ich das gerne jedes mal wenn long press gecalled wird und beim game tile hat sich die vibration falsch angefühlt. check das gerne nochmal ob das auf android auch passt
sneeex added 2 commits 2026-05-13 13:38:27 +00:00
remove tooltip attribute f87a873362
Merge remote-tracking branch 'origin/feature/215-haptisches-feedback-hinzufügen' into feature/215-haptisches-feedback-hinzufügen
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 47s
Details
Pull Request Pipeline / lint (pull_request) Successful in 54s
Details
cadb5793e7
sneeex commented 2026-05-13 13:45:50 +00:00
Author
Owner
Copy Link

Hab bei zwei long presses das Feedback überarbeitet, beim Live Edit wollte ich das gerne jedes mal wenn long press gecalled wird und beim game tile hat sich die vibration falsch angefühlt. check das gerne nochmal ob das auf android auch passt

passt

> Hab bei zwei long presses das Feedback überarbeitet, beim Live Edit wollte ich das gerne jedes mal wenn long press gecalled wird und beim game tile hat sich die vibration falsch angefühlt. check das gerne nochmal ob das auf android auch passt passt
sneeex added 1 commit 2026-05-13 13:55:40 +00:00
remove space
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 46s
Details
Pull Request Pipeline / lint (pull_request) Successful in 54s
Details
0d497236b0
sneeex added 1 commit 2026-05-13 14:31:26 +00:00
disable button splash
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 48s
Details
Pull Request Pipeline / lint (pull_request) Successful in 55s
Details
f5b24cb923
sneeex requested review from flixcoo 2026-05-13 14:31:36 +00:00
flixcoo approved these changes 2026-05-14 13:08:54 +00:00
flixcoo merged commit 7f024534aa into development 2026-05-14 13:09:01 +00:00
flixcoo deleted branch feature/215-haptisches-feedback-hinzufügen 2026-05-14 13:09:01 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
flixcoo
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
In Review
1
Milestone
No items
No Milestone Alpha
Assignees
Clear assignees
actions flixcoo gelbeinhalb sneeex
No Assignees sneeex
3 Participants
Notifications
Total Time Spent: 10 minutes
flixcoo
10 minutes
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: liquid-development/game-tracker#216
Delete Branch "feature/215-haptisches-feedback-hinzufügen"

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?