entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 36 Releases Activity

misc: adjust map behaviours and controls (#41466) #681

Merged
fpeters merged 1 commits from wip/41466-map-zoom-gestures into main 2023-09-19 16:54:09 +02:00
Conversation 1 Commits 1 Files Changed 2 +8 -16
fpeters commented 2023-09-16 11:41:34 +02:00
Owner
Copy Link
No description provided.
fpeters added 1 commit 2023-09-16 11:41:34 +02:00
gitea/wcs/pipeline/head This commit looks good Details
889c0fc933 misc: adjust map behaviours and controls (#41466) 
* always enable gesture controls
* always requires Ctrl to zoom
* do not include search button on readonly maps
* do not include zoom buttons on small maps
fpeters commented 2023-09-16 11:43:03 +02:00
Author
Owner
Copy Link

J'ai mis dans https://dev.entrouvert.org/issues/41466#note-2 du contexte sur ce que j'ai décidé de faire ici, mais je mets une copie ici aussi :

J'ai remonté le changement de comportement, à la base c'était #25828 ("Ne pas permettre le déplacement dans la carte sur la page de validation") qui demandait à ne pas pouvoir déplacer une carte en lecture seule (sur la page de validation.

Mais ça laisse le zoom, et si on active le mode "gestures", pour avoir le message "vous pouvez zoomer avec ctrl-molette", ça réétablit la possibilité de se déplacer dans la carte.

Je décide de tout reprendre pour avoir :

  • tout le temps la possibilité de zoomer/déplacer, avec le mode "gestures" activé pour avoir le message,
  • de ne pas inclure le bouton de recherche sur les cartes lecture seule,
  • de ne pas inclure les boutons de zoom sur les petites cartes (dans le tableau de traitement ou en barre latérale).

Alternativement, pour répondre strictement à ce ticket, la modification aurait été de faire la désactivation totale zoom/déplacement/gesture en visant uniquement le cas "page de validation", mais j'ai trouvé que ça faisait bizarre, et que la demande initiale ne justifiait pas d'avoir juste un cas particulier.

J'ai mis dans https://dev.entrouvert.org/issues/41466#note-2 du contexte sur ce que j'ai décidé de faire ici, mais je mets une copie ici aussi : J'ai remonté le changement de comportement, à la base c'était #25828 ("Ne pas permettre le déplacement dans la carte sur la page de validation") qui demandait à ne pas pouvoir déplacer une carte en lecture seule (sur la page de validation. Mais ça laisse le zoom, et si on active le mode "gestures", pour avoir le message "vous pouvez zoomer avec ctrl-molette", ça réétablit la possibilité de se déplacer dans la carte. Je décide de tout reprendre pour avoir : * tout le temps la possibilité de zoomer/déplacer, avec le mode "gestures" activé pour avoir le message, * de ne pas inclure le bouton de recherche sur les cartes lecture seule, * de ne pas inclure les boutons de zoom sur les petites cartes (dans le tableau de traitement ou en barre latérale). Alternativement, pour répondre strictement à ce ticket, la modification aurait été de faire la désactivation totale zoom/déplacement/gesture en visant uniquement le cas "page de validation", mais j'ai trouvé que ça faisait bizarre, et que la demande initiale ne justifiait pas d'avoir juste un cas particulier.
csechet approved these changes 2023-09-19 16:42:02 +02:00
fpeters merged commit eb0ac023c6 into main 2023-09-19 16:54:09 +02:00
fpeters deleted branch wip/41466-map-zoom-gestures 2023-09-19 16:54:09 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
csechet
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#681
No description provided.
Delete Branch "wip/41466-map-zoom-gestures"

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?