Skip to content

Show hide response#8

Open
Raserber wants to merge 8 commits into
mainfrom
showHide-response
Open

Show hide response#8
Raserber wants to merge 8 commits into
mainfrom
showHide-response

Conversation

@Raserber

Copy link
Copy Markdown

proposition pour résolution : #7

Fixes #7 Add ability show/hide card response
Fixes #7 update YAML files for show/hide ability
@Raserber Raserber requested a review from Rowin January 28, 2026 09:36
Fixes #7 : undo localStorage of showResponse in GameCore.vue

@Rowin Rowin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Il y a un petit problème de conception pour moi, qui explique les difficultés pour synchroniser showResponse partout dans l'application

Pour moi il devrait y avoir un paramètre "showResponse" global, défini au niveau de l'application et lu depuis le localStorage, puis un autre paramètre au niveau de la carte affichée, qu'on peut override depuis la carte.

Si le global est désactivé : les réponses ne sont pas affichées par défaut, mais sur chaque carte je peux choisir d'afficher ponctuellement la réponse pour la carte en cours en cliquant sur le bouton

Si le global est activé, les réponses sont affichées par défaut, mais sur chaque carte je peux choisir de masquer ponctuellement la réponse pour la carte en cours en cliquant sur le bouton (je vois moins le cas d'usage de masquer la réponse ponctuellement, surtout qu'elle serait déjà affichée, mais pourquoi pas)

Cette manière de faire permet de centraliser les lectures/écritures du localStorage dans App.vue et évite d'avoir à resynchroniser l'état lorsqu'on ouvre la sidebar, ce qui est un peu antinomique avec la réactivité de Vue

Comment thread public/malaise-dating.yaml Outdated
Comment thread src/GameCore.vue Outdated
Comment thread src/GameCore.vue Outdated
@Rowin Rowin linked an issue Jan 28, 2026 that may be closed by this pull request
@Raserber Raserber requested a review from Rowin January 30, 2026 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Mode présentation

2 participants