includes: style search control of map widgets (#73852) #143

csechet · 2023-01-26T13:05:37+01:00

csechet commented

2023-01-26 13:05:37 +01:00

No description provided.

fpeters commented

2023-01-30 14:08:45 +01:00

Je pense qu'il faudait cursor: pointer au-dessus des choix; j'aurais bien vu aussi un souligné sur le survol mais vu que ce sont des <option> je ne suis pas sûr que ça soit possible. (j'ai un peu peur de ce choix de balisage)

Je pense qu'il faudait cursor: pointer au-dessus des choix; j'aurais bien vu aussi un souligné sur le survol mais vu que ce sont des `<option>` je ne suis pas sûr que ça soit possible. (j'ai un peu peur de ce choix de balisage)

fpeters referenced this pull request from entrouvert/wcs

2023-01-30 14:09:15 +01:00

widgets: add a search control to map widget (#72558) #67

csechet commented

2023-01-30 15:11:04 +01:00

Le pointeur et le souligné au survol sont possible, étant donné qu'un "appaearance: none" est posé sur les select plus haut dans le CSS.

Les avantages d'utiliser un select / options plutôt que des divs sont :

L'accessibilité : ça évite d'avoir à gérer les attributs Aria et les comportements de l'utilisation au clavier (C.F https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/listbox_role). Notamment l'ordre de focus n'est pas censé passer le focus à chaque item de la liste lors d'un appui sur Tab, ce qui devient pénible à gérer avec des liens.
Économiser beaucoup de CSS.
Garder une consistance avec les thèmes existant, et éviter d'avoir à y penser lors des intégration graphique futures : si on ajoute un style pour cette liste, sans les variables scss avec la sémantique qu'il faut et qui n'existent pas à ma connaissance, on va se retrouver avec des versions variables de listes sélectionnables. C'est déjà le cas avec les autocomplete et listes issues de select2 je crois. Ça serait probablement une bonne idée de créer ces variables cependant.

Dis moi si tu tiens à utiliser des divs avec des roles aria, je reprendrai les patchs.

Le pointeur et le souligné au survol sont possible, étant donné qu'un "appaearance: none" est posé sur les select plus haut dans le CSS. Les avantages d'utiliser un select / options plutôt que des divs sont : * L'accessibilité : ça évite d'avoir à gérer les attributs Aria et les comportements de l'utilisation au clavier (C.F https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/listbox_role). Notamment l'ordre de focus n'est pas censé passer le focus à chaque item de la liste lors d'un appui sur Tab, ce qui devient pénible à gérer avec des liens. * Économiser beaucoup de CSS. * Garder une consistance avec les thèmes existant, et éviter d'avoir à y penser lors des intégration graphique futures : si on ajoute un style pour cette liste, sans les variables scss avec la sémantique qu'il faut et qui n'existent pas à ma connaissance, on va se retrouver avec des versions variables de listes sélectionnables. C'est déjà le cas avec les autocomplete et listes issues de select2 je crois. Ça serait probablement une bonne idée de créer ces variables cependant. Dis moi si tu tiens à utiliser des divs avec des roles aria, je reprendrai les patchs.

csechet referenced this pull request from entrouvert/wcs

2023-01-30 15:12:36 +01:00

widgets: add a search control to map widget (#72558) #67

csechet force-pushed wip/73852-styler-le-controle-de-recherche- from fe91c04001 to 5997655cb8

2023-02-16 20:06:12 +01:00

Compare

csechet merged commit 5c09ef9209 into main

2023-02-16 20:42:08 +01:00

csechet deleted branch wip/73852-styler-le-controle-de-recherche-

2023-02-16 20:42:08 +01:00

Sign in to join this conversation.

No reviewers

No Label

No Milestone

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/publik-base-theme#143