Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Styleguide: mise à jour du breadcrumb et de la navigation mobile #718
Styleguide: mise à jour du breadcrumb et de la navigation mobile #718
Changes from 18 commits
0f93b1b
8afb129
6225c1f
33ab348
2df37f2
4aaddf4
937ed07
aa8ec51
0c31177
8c30718
0075894
9775346
4ec7032
a726cc7
a8782b7
1529e32
26c6487
1143146
8b7aa4a
f220a78
c0b2778
16f2e09
8cb26bb
43a4675
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dans certains cas, le dropdown n'est pas complètement visible :
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ce point là est assez délicat et nécessite un peu de js. J'ai tenté la chose suivante: au clic sur le bouton, on calcule la position du bouton et la largeur du dropdown. Si le dropdown est plus large que l'espace disponible à droite du bouton, on ajoute une classe qui permet de décaler le dropdown sur la gauche. J'ai testé plusieurs cas de figure, et ça a l'air de fonctionner comme voulu.
(à voir si le javascript peut être amélioré, ce n'est pas du tout mon domaine d'expertise)
Seul cas qui reste problématique: quand le breadcrumb item passe sur deux lignes, le dropdown le recouvre partiellement (et recouvre le bouton). On peut quand même le fermer en cliquant n'importe où en dehors du dropdown. Et idéalement, il ne faudrait pas avoir des éléments aussi longs dans un breadcrumb…
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pour moi, c'est tout bon comme ça. (j'ai juste formaté le js).
Pour le comportement sur deux lignes, il faut confirmer avec l'équipe WordPress si c'est ok pour eux.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Il y a effectivement des choses qui pourraient être plus jolies le comportement sur deux lignes, comme le sous-menu laissant apparaître le bouton, ou encore les éléments s'affichant comme un texte suivi (cf. ci-dessous, mais je ne vois pas comment faire avec un
<li>
).Mais là, on parle du 1% du 99% du travail déjà effectué. Je valide la PR, bon boulot!