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

misc: strip emojis from buttons (#76405) #228

Merged
fpeters merged 1 commits from wip/76405-a11y-emoji-buttons into main 2023-04-14 10:54:47 +02:00
Conversation 0 Commits 1 Files Changed 4 +34 -3
fpeters commented 2023-04-07 17:00:06 +02:00
Owner
Copy Link
No description provided.
fpeters force-pushed wip/76405-a11y-emoji-buttons from 17e867a51d to 2a8f9fc6e3 2023-04-07 17:19:50 +02:00 Compare
fpeters force-pushed wip/76405-a11y-emoji-buttons from 2a8f9fc6e3 to 1a7d5fbc6a 2023-04-07 18:11:39 +02:00 Compare
fpeters reviewed 2023-04-08 11:06:37 +02:00
debian/control Outdated
@ -22,6 +22,7 @@ Depends: graphviz,
python3-django-ckeditor,
python3-django-ratelimit,
python3-dnspython,
python3-emoji,
fpeters commented 2023-04-08 11:03:14 +02:00
Author
Owner
Copy Link

Le module est packagé pour bookworm : https://packages.debian.org/bookworm/python3-emoji et le paquet s'installe et fonctionne sur buster. J'ai hésité à faire sans et juste utiliser une regex un peu large ou moche mais comme c'était disponible, autant l'exploiter.

Aussi, ça permettrait ailleurs de fournir un filtre de remplacement (et/ou suppression) des emojis, qui pourrait être utilisé lors d'envoi vers des systèmes qui plantent sur ces caractères (par exemple https://dev.entrouvert.org/issues/75166, ou https://dev.entrouvert.org/issues/54028 précédemment).

Le module est packagé pour bookworm : https://packages.debian.org/bookworm/python3-emoji et le paquet s'installe et fonctionne sur buster. J'ai hésité à faire sans et juste utiliser une regex un peu large ou moche mais comme c'était disponible, autant l'exploiter. Aussi, ça permettrait ailleurs de fournir un filtre de remplacement (et/ou suppression) des emojis, qui pourrait être utilisé lors d'envoi vers des systèmes qui plantent sur ces caractères (par exemple https://dev.entrouvert.org/issues/75166, ou https://dev.entrouvert.org/issues/54028 précédemment).
tests/test_widgets.py Outdated
@ -1412,0 +1429,4 @@
# single emoji (do not do this) (no empty aria-label)
form = Form(use_tokens=False)
form.add_submit('submit', '✅')
fpeters commented 2023-04-08 11:06:32 +02:00
Author
Owner
Copy Link

Il y aurait à communiquer des bonnes pratiques sur l'utilisation des emojis dans les boutons, au moins :

  • celle notée ici de ne pas juste utiliser un emoji,
  • mais aussi déconseiller les emojis dont les couleurs ne fonctionnent pas sur fond sombre, cf l'exemple donné en lien dans le ticket, "Les emojis ➕ et ➖ sont peu visibles sur fond sombre."
Il y aurait à communiquer des bonnes pratiques sur l'utilisation des emojis dans les boutons, au moins : * celle notée ici de ne pas juste utiliser un emoji, * mais aussi déconseiller les emojis dont les couleurs ne fonctionnent pas sur fond sombre, cf l'exemple donné en lien dans le ticket, "Les emojis ➕ et ➖ sont peu visibles sur fond sombre."
fpeters changed title from WIP: misc: strip emojis from buttons (#76405) to misc: strip emojis from buttons (#76405) 2023-04-08 11:06:43 +02:00
tnoel requested changes 2023-04-10 19:52:05 +02:00
wcs/qommon/form.py Outdated
@ -211,1 +212,3 @@
value = htmlescape(self.label) if self.label else None
label = self.label or ''
if label and 'aria-label' not in self.attrs:
cleaned_label = emoji.replace_emoji(label).strip()
tnoel commented 2023-04-10 19:51:59 +02:00
Owner
Copy Link

J'ai dû aller voir la doc pour comprendre que le replace en fait efface les emojis. Je serais pour qu'on ajoute ici la valeur par défault replace='' dans l'appel


cleaned_label = emoji.replace_emoji(label, replace='').strip()
J'ai dû aller voir la doc pour comprendre que le replace en fait efface les emojis. Je serais pour qu'on ajoute ici la valeur par défault replace='' dans l'appel ``` cleaned_label = emoji.replace_emoji(label, replace='').strip() ```
fpeters commented 2023-04-10 20:10:35 +02:00
Author
Owner
Copy Link

Voilà.

Voilà.
fpeters force-pushed wip/76405-a11y-emoji-buttons from 1a7d5fbc6a to a5fbddeced 2023-04-10 20:07:30 +02:00 Compare
fpeters force-pushed wip/76405-a11y-emoji-buttons from a5fbddeced to 2e8d588777 2023-04-10 20:10:22 +02:00 Compare
fpeters requested review from tnoel 2023-04-11 07:47:02 +02:00
tnoel approved these changes 2023-04-14 10:53:52 +02:00
fpeters merged commit c40f5a22ec into main 2023-04-14 10:54:47 +02:00
fpeters deleted branch wip/76405-a11y-emoji-buttons 2023-04-14 10:54:47 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
tnoel
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#228
No description provided.
Delete Branch "wip/76405-a11y-emoji-buttons"

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?