Utilisation de boutons DSFR dans le déplacement de RDV #4941
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
victormours
reviewed
Jan 6, 2025
| span.d-md-none.ml-1< | ||
| | sem. précédente | ||
| = link_to "", creneaux_users_rdv_path(@rdv, date: previous_from_date), class: "fr-btn fr-icon-arrow-left-s-line fr-hidden fr-unhidden-md", aria: { label: "Semaine précédente" } | ||
| = link_to "sem. précédente", creneaux_users_rdv_path(@rdv, date: previous_from_date), class: "fr-btn fr-btn--icon-left fr-icon-arrow-left-s-line fr-hidden-md fr-hidden-lg", aria: { label: "Semaine précédente" } |
There was a problem hiding this comment.
le fr-hidden-lg n'est pas utile ici, puis que fr-hidden-md cache à partir du breakpoint md
En fait je me demande si on pourrait pas avoir une écriture plus simple et plus similaire à du responsive "classique" avec un truc du genre :
= link_to creneaux_users_rdv_path(@rdv, date: previous_from_date), class: "fr-btn fr-icon-arrow-left-s-line", aria: { label: "Semaine précédente" } do
.fr-hidden-md = "sem. précédente"
et comme ça on a un seul link_to
There was a problem hiding this comment.
C’est une proposition intéressante, mais malheureusement ça ne passe pas, car on a des class différentes entre le bouton icône seul et le bouton icône + texte
Je pense qu’on peut rester sur ça dans un premier temps et revenir dessus + tard
AntoineGirard
force-pushed
the
agd/creneau-modif-use-dsfr-button
branch
2 times, most recently
from
34606e4 to
99774dc
Compare
AntoineGirard
force-pushed
the
agd/creneau-modif-use-dsfr-button
branch
from
99774dc to
3fc385c
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Contexte
Comme cela a été fait pour la recherche de RDV, on utilise des boutons DSFR pour le déplacement de RDV.
Voir #4929
Solution
Note
Ici on a pris le parti de changer uniquement les boutons sans revoir l’UX. Des propositions sont en train d’être étudiées pour améliorer l’UX de la sélection de créneau. Cela sera traité dans d’autres PR.
Captures d'écran