entrouvert
/
publik-base-theme
15
1
Fork 1
Code Issues Pull Requests 10 Releases Wiki Activity

css: style for acked notifications in notifications cell (#78314) #309

Merged
lguerin merged 1 commits from wip/78314-notification-acked into main 2024-04-13 12:14:03 +02:00
Conversation 5 Commits 1 Files Changed 1 +6
lguerin commented 2023-06-09 18:06:23 +02:00
Owner
Copy Link
No description provided.
lguerin added 1 commit 2023-06-09 18:06:24 +02:00
gitea/publik-base-theme/pipeline/head This commit looks good Details
f269d74cb1
css: style for acked notifications in notifications cell (#78314)
lguerin commented 2023-06-09 18:07:16 +02:00
Author
Owner
Copy Link

Exemple avec le thème clapotis les canards.

Je ne trouve pas ça ultra flagrant, peut-être mettre un peu plus d'opacité ?

Exemple avec le thème clapotis les canards. Je ne trouve pas ça ultra flagrant, peut-être mettre un peu plus d'opacité ?
notifs.png
8.3 KiB
notifs.png
tjund commented 2023-06-12 10:10:05 +02:00
Owner
Copy Link

Niveau a11y on se retrouve avec un contraste insufisant.
Est-ce que le texte d'une notif déjà lu peu devenir illisible ?
Supprimer le pointeur sur le lien, je ne sais pas non plus si c'est une bonne idée (j'ai un doute, puisque ça reste un lien ?)
Est-ce qu'il ne faudrait pas trouver une autre astuce graphique, compatible rgaa pour différencier lu / non lu ?

Niveau a11y on se retrouve avec un contraste insufisant. Est-ce que le texte d'une notif déjà lu peu devenir illisible ? Supprimer le pointeur sur le lien, je ne sais pas non plus si c'est une bonne idée (j'ai un doute, puisque ça reste un lien ?) Est-ce qu'il ne faudrait pas trouver une autre astuce graphique, compatible rgaa pour différencier lu / non lu ?
lguerin commented 2023-06-13 14:41:11 +02:00
Author
Owner
Copy Link

@tjund je ne fais que reporter ce qu'on trouve dans 2 ou 3 thèmes, mais je veux bien que tu fasses une meilleure proposition si tu en as une :)

@tjund je ne fais que reporter ce qu'on trouve dans 2 ou 3 thèmes, mais je veux bien que tu fasses une meilleure proposition si tu en as une :)
csechet added 1 commit 2023-08-29 15:18:27 +02:00
gitea/publik-base-theme/pipeline/head This commit looks good Details
cec991a566 sinon on fait l'inverse
csechet requested review from tjund 2023-11-10 14:39:08 +01:00
lguerin commented 2024-02-08 16:52:08 +01:00
Author
Owner
Copy Link

@tjund qu'en penses-tu ?

@tjund qu'en penses-tu ?
tjund requested changes 2024-03-18 16:19:44 +01:00
Dismissed
static/includes/_cells.scss Outdated
@ -833,0 +833,4 @@
div.notifications-cell li.combo-notification:not(.combo-notification-acked) {
&, & > a {
font-weight: bold;
tjund commented 2024-03-18 16:19:41 +01:00
Owner
Copy Link

plutôt "bolder" pour rester pertinent avec les thèmes qui ont mis du bold dans les items de liste de liens ?

plutôt "bolder" pour rester pertinent avec les thèmes qui ont mis du bold dans les items de liste de liens ?
lguerin commented 2024-03-22 10:54:36 +01:00
Author
Owner
Copy Link

fait

fait
tjund commented 2024-03-18 16:20:15 +01:00
Owner
Copy Link

Qu'à se pencher sur le sujet, j'aurais aimé une solution qui :

  • Niveau a11y retourne une information que le message est nouveau
  • Niveau graphique fonctionne avec une très grande majorité de thèmes.

Mais comme je n'ai rien fait et que la propostion de Corentin va dans le bon sens. Partons là dessus pour le moment. Je propose juste de remplacer "bold" par "bolder"

Qu'à se pencher sur le sujet, j'aurais aimé une solution qui : * Niveau a11y retourne une information que le message est nouveau * Niveau graphique fonctionne avec une très grande majorité de thèmes. Mais comme je n'ai rien fait et que la propostion de Corentin va dans le bon sens. Partons là dessus pour le moment. Je propose juste de remplacer "bold" par "bolder"
lguerin force-pushed wip/78314-notification-acked from cec991a566 to d4ca2926b5 2024-03-22 10:54:24 +01:00 Compare
lguerin requested review from tjund 2024-03-22 10:54:40 +01:00
tjund approved these changes 2024-04-03 12:08:07 +02:00
lguerin force-pushed wip/78314-notification-acked from d4ca2926b5 to 41ad6b323b 2024-04-13 12:11:33 +02:00 Compare
lguerin merged commit 41ad6b323b into main 2024-04-13 12:14:03 +02:00
lguerin deleted branch wip/78314-notification-acked 2024-04-13 12:14:03 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
tjund
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 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#309
No description provided.
Delete Branch "wip/78314-notification-acked"

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?