feat: adds possibility to copy coordinates on clipboard#166
feat: adds possibility to copy coordinates on clipboard#166QuentinMadura merged 5 commits intodevfrom
Conversation
| cy.contains("Plantabilité élevée") | ||
| }) | ||
|
|
||
| it("MapPopUp when selectedDataType is LOCAL_CLIMATE_ZONES", () => { |
There was a problem hiding this comment.
J'ai supprimé l'un des deux tests parce qu'ils vérifient la même chose
There was a problem hiding this comment.
Le format des PopUp selon la source de données n'est pas le même, et le traitement à partir de la donnée récupérée index non plus.
Je crois que ça servrait aussi à tester que la popup correspondait bien à la source
| @@ -9,25 +9,7 @@ describe("MapscorePopup Component", () => { | |||
| beforeEach(() => { | |||
| setActivePinia(createPinia()) | |||
| }) | |||
There was a problem hiding this comment.
Pas de possibilité d'ajouter un test ici parce que le fait d'utiliser la fonction navigator.clipboard.writeText(value) lève une erreur au niveau de cypress. C'est bien documenté ici et ça doit être résolu par eux
|
closes #82 |
ludovicdmt
left a comment
ludovicdmt
left a comment
There was a problem hiding this comment.
Pas sur pour le test que tu as viré
| cy.contains("Plantabilité élevée") | ||
| }) | ||
|
|
||
| it("MapPopUp when selectedDataType is LOCAL_CLIMATE_ZONES", () => { |
There was a problem hiding this comment.
Le format des PopUp selon la source de données n'est pas le même, et le traitement à partir de la donnée récupérée index non plus.
Je crois que ça servrait aussi à tester que la popup correspondait bien à la source
| @@ -9,25 +9,7 @@ describe("MapscorePopup Component", () => { | |||
| beforeEach(() => { | |||
| setActivePinia(createPinia()) | |||
| }) | |||
Marc-AntoineA
left a comment
Marc-AntoineA
left a comment
There was a problem hiding this comment.
Ça manque vraiment d’une animation / message pour dire "copié dans le presse papier".
Là, on ne comprend pas qu’on a fait qqch.
Je peux m’attendre à avoir les coordonnées qui sont remplacées par un "copié dans le press-papier".
On peut aussi imaginer que au "hover sur les coordonnées", le texte est remplacé par du "cliquer pour copier".
Sinon un message qui apparaît sur l’écran…
ludovicdmt
left a comment
ludovicdmt
left a comment
There was a problem hiding this comment.
Ajout de message au clic et au survol (cf Commentaire dans le code)
| /> | ||
| </svg> | ||
| </button> | ||
| </div> |
There was a problem hiding this comment.
Au niveau UX il manque pour moi :
- Un message "Copié!" qui apparaît au clic
- Un message "Copier les coordonées" qui apparaît au survol (pour indiquer plus clairement ce que va faire le clic)
There was a problem hiding this comment.
J'ai ajouté (en quick-win) un "Copié" au clic qui je dois l'avouer n'est pas très satisfaisant.
En terme d'améliorations (mais pas maintenant), je verrai :
- Lorsque l'on clique, un toast s'affiche pour nous indiquer qu'on a bien copié les coordonnées
- Au survol, un tooltip apparaît pour nous indiquer "copier les coordonnées"
Qu'en penses-tu @ludovicdmt ?
There was a problem hiding this comment.
Je viens de voir ça et de le tester.
Moi ça me convient en l'état et que je rédige un ticket pour les améliorations plus tard.
Merci Quentin !
Ah bah j'avais pas vu ton message mais on est sur la meme ligne ! |
|
#169 restera à traiter dans une prochaine PR |
3 participants
Resolve #82