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

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

Open
lguerin wants to merge 2 commits from wip/78314-notification-acked into main
pull from: wip/78314-notification-acked
merge into: entrouvert:main
Conversation 3 Commits 2 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
All checks were successful
gitea/publik-base-theme/pipeline/head This commit looks good
Details
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b wip/78314-notification-acked main
git pull origin wip/78314-notification-acked

Step 2:

Merge the changes and update on Gitea.
git checkout main
git merge --no-ff wip/78314-notification-acked
git push origin main
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?