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

a11y: switch top links to ul/li (#40926) #256

Merged
fpeters merged 1 commits from wip/40926-user-info-as-list into main 2023-05-12 09:19:21 +02:00
Conversation 2 Commits 1 Files Changed 25 +181 -131
fpeters commented 2023-05-07 13:29:05 +02:00
Owner
Copy Link

C'est moins intrusif que ce qui pouvait être imaginé, ça reste néanmoins à plutôt appliquer en début de cycle pour permettre au maximum d'identifier des intégrations graphiques qui poseraient problème.

C'est moins intrusif que ce qui pouvait être imaginé, ça reste néanmoins à plutôt appliquer en début de cycle pour permettre au maximum d'identifier des intégrations graphiques qui poseraient problème.
tjund commented 2023-05-09 10:53:20 +02:00
Owner
Copy Link

Je n'ai pas compris ce que devient la balise span.sep, ou comment est géré le séparateur pour les thèmes qui l'utilisent (même si ça me va bien si cette balise disparait)

Je n'ai pas compris ce que devient la balise span.sep, ou comment est géré le séparateur pour les thèmes qui l'utilisent (même si ça me va bien si cette balise disparait)
fpeters commented 2023-05-09 11:18:11 +02:00
Author
Owner
Copy Link

Je n'ai pas compris ce que devient la balise span.sep, ou comment est géré le séparateur pour les thèmes qui l'utilisent (même si ça me va bien si cette balise disparait)

Ce n'est plus cette classe depuis un certain temps (elle avait été supprimée dans #63860, j'avais du la remettre dans #64550), elle s'appelle désormais .user-info--separator, et donc pour l'exemple bethune-bruay,

-.user-info--separator::after {
+.toplinks--list-item:not(:last-child)::after {
        content: "·";
 }

et oui ça fait apparaitre le séparateur également dans la situation où l'usager est connecté mais c'est de toute façon quelque chose de souhaitable.

> Je n'ai pas compris ce que devient la balise span.sep, ou comment est géré le séparateur pour les thèmes qui l'utilisent (même si ça me va bien si cette balise disparait) Ce n'est plus cette classe depuis un certain temps (elle avait été supprimée dans #63860, j'avais du la remettre dans #64550), elle s'appelle désormais .user-info--separator, et donc pour l'exemple bethune-bruay, ``` -.user-info--separator::after { +.toplinks--list-item:not(:last-child)::after { content: "·"; } ``` et oui ça fait apparaitre le séparateur également dans la situation où l'usager est connecté mais c'est de toute façon quelque chose de souhaitable.
tjund requested changes 2023-05-10 11:33:43 +02:00
static/includes/_user-info.scss Outdated
@ -52,0 +53,4 @@
.toplinks--list {
margin: 0;
padding: 0;
list-style: none;
tjund commented 2023-05-10 11:31:33 +02:00
Owner
Copy Link

Le thème cinor-2021 ajoute un icon via #toplinks::before

Pour éviter un passage à la ligne, il faudrait passer --list en inline-block

display: inline-block

Le thème cinor-2021 ajoute un icon via `#toplinks::before` Pour éviter un passage à la ligne, il faudrait passer --list en inline-block `display: inline-block`
fpeters commented 2023-05-10 12:14:44 +02:00
Author
Owner
Copy Link

Pour éviter un passage à la ligne, il faudrait passer --list en inline-block

Ça pétera d'autres intégrations graphiques sur l'alignement, ça ne peut pas être fait ainsi.

> Pour éviter un passage à la ligne, il faudrait passer --list en inline-block Ça pétera d'autres intégrations graphiques sur l'alignement, ça ne peut pas être fait ainsi.
fpeters commented 2023-05-10 12:30:22 +02:00
Author
Owner
Copy Link

J'ai adapté cinor-2021.

J'ai adapté cinor-2021.
fpeters marked this conversation as resolved
templates/includes/user-info.html Outdated
@ -24,0 +7,4 @@
<li class="toplinks--list-item">
{% if account_url %}<a class="account-link" href="{{ account_url }}">{% endif %}
<span class="connected-user">{% block user-info-user-name %}{% include "includes/user-info-user-name.html" %}{% endblock %}</span>{% if account_url %}</a>{% endif %}
</li><li class="toplinks--list-item">
tjund commented 2023-05-10 11:33:29 +02:00
Owner
Copy Link

L'indentation du fichier est étonnant, c'est pour éviter les espaces mot non maitrisés ?

L'indentation du fichier est étonnant, c'est pour éviter les espaces mot non maitrisés ?
fpeters commented 2023-05-10 12:15:21 +02:00
Author
Owner
Copy Link

L'indentation est automatique via djhtml, ça n'est pas toujours idéal mais ça n'en fait plus un sujet de discussion.

L'indentation est automatique via djhtml, ça n'est pas toujours idéal mais ça n'en fait plus un sujet de discussion.
fpeters marked this conversation as resolved
templates/includes/user-info.html Outdated
@ -24,0 +20,4 @@
</a>
</li><li class="toplinks--list-item">
<a class="registration" href="{{registration_url}}"
>{% block user-info-registration-label %}{% include "includes/user-info-registration-label.html" %}{% endblock %}{% endif %}</a></span>
tjund commented 2023-05-10 11:28:17 +02:00
Owner
Copy Link

reste un à supprimer

reste un </span> à supprimer
tjund commented 2023-05-10 11:28:40 +02:00
Owner
Copy Link

</span>

`</span>`
fpeters commented 2023-05-10 12:30:34 +02:00
Author
Owner
Copy Link

J'ai supprimé ce span perdu.

J'ai supprimé ce span perdu.
fpeters marked this conversation as resolved
fpeters force-pushed wip/40926-user-info-as-list from 3a7698ab8e to aa3d6cc3ff 2023-05-10 12:30:01 +02:00 Compare
fpeters requested review from tjund 2023-05-10 12:30:49 +02:00
tjund approved these changes 2023-05-10 14:25:37 +02:00
tjund left a comment
Owner
Copy Link

Nickel, beau patch.

Nickel, beau patch.
fpeters merged commit f29730f88a into main 2023-05-12 09:19:21 +02:00
fpeters deleted branch wip/40926-user-info-as-list 2023-05-12 09:19:21 +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
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#256
No description provided.
Delete Branch "wip/40926-user-info-as-list"

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?