Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(drawing): better UX #328

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

feat(drawing): better UX #328

wants to merge 8 commits into from

Conversation

azarz
Copy link
Member

@azarz azarz commented Jan 9, 2025

fixes #312

(en partie, création d'issues à venir pour le reste, qu'on fera plus tard)

@azarz azarz self-assigned this Jan 9, 2025
@elias75015
Copy link
Contributor

En mode DSFR, même genre de problème que pour le Layerswitcher en positionnement absolu.
Pour rappel :
#319
résolu par :
#324

@azarz
Copy link
Member Author

azarz commented Jan 10, 2025

Vu, je règle ça

@elias75015
Copy link
Contributor

Yaurait pas moyen de rajouter un label avant le picto du style "Couleur du pictogramme" ?
image

Le fait qu'il faille cliquer sur le picto pour changer sa couleur n'est pas hyper intuitif

@azarz
Copy link
Member Author

azarz commented Jan 10, 2025

C'est corrigé pour le 2 points @elias75015

@elias75015
Copy link
Contributor

Cool ! Parfait.

Désolé de t'embeter encore, j'ai repéré deux petits problèmes sur les features textuels du drawing (le comportement est le même sur la main). Je pense que ca peut être l'occasion de corriger ça.

1 - la pop-up n'a pas de bouton enregistrer contrairement à toutes les autres pop-up. En mode classique passe, mais en mode dsfr je ne crois pas que ca puisse le faire.

image

Au moins (ou uniquement si c'est plus simple) en mode DSFR, ajouter le bouton enregistrer

2 - il n'y a pas d'unicité sur les pop-ups avec contrairement aux autres. Sinon on ne valide pas le texte saisi et qu'on clique ailleurs, la pop-up reste affichée en l'état.

Si c'est pas trop galère, inclure une unicité sur ces pop-up de validation, comme pour les autres (point, polygone, polyligne)
image
image

@azarz
Copy link
Member Author

azarz commented Jan 10, 2025

Corrigé pour le 2 points @elias75015

@elias75015
Copy link
Contributor

quand tu cliques sur enregistrer, et que tu recrées un texte dans la foulée, le texte que tu as enregistré disparait
Capture vidéo du 10-01-2025 17:12:26.webm

@azarz
Copy link
Member Author

azarz commented Jan 10, 2025

corrigé

Copy link
Contributor

@elias75015 elias75015 left a comment

Choose a reason for hiding this comment

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

🥳 et bon week end

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

refacto(drawing): evolution sur le style du widget drawing en mode DSFR
2 participants