Skip to content
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

Migration des icônes RDV usager au DSFR #4939

Merged
merged 3 commits into from
Jan 6, 2025

Conversation

AntoineGirard
Copy link
Member

@AntoineGirard AntoineGirard commented Jan 2, 2025

Contexte

On continue à se débarasser des icônes font awesome dans l’interface usager.

Solution

  • On change tous les icônes
  • On en profite pour remplacer un bouton Bootstrap par un bouton DSFR
  • On utilise des badges DSFR pour les tags « À venir » et « Annulé »

À discuter dans une autre PR avec le trio, est-ce qu’on n'afficherait pas des intitulés plutôt qu'uniquement les pictos ?

Captures d'écran

Avant Après
image image
image image

@AntoineGirard AntoineGirard force-pushed the agd/migrate-usager-rdv-icons-dsfr branch from b22b9ad to 524a943 Compare January 2, 2025 17:21
Copy link
Contributor

@adipasquale adipasquale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ nickel merci beaucoup, rien à redire.

parfait cette taille de PR 🙇

@@ -2,42 +2,42 @@

ul.list-group.fr-mb-3w
li.list-group-item
i.fa.fa-info-circle>
span.fr-icon.fr-icon-info-fill.fr-mr-1w[aria-hidden="true"]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pour les motifs côté agents dans le menu de gauche, on a commencé à utiliser .fr-icon-draft-fill, peut-être qu'on peut faire ça ici aussi (voir #4916)

| Ce RDV se déroulera à domicile

- elsif @rdv.public_office?
li.list-group-item
.fa.fa-map-marker-alt>
span.fr-icon.fr-icon-map-pin-2-fill.fr-mr-1w[aria-hidden="true"]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Et pour les lieux dans le menu de gauche, on utilise .fr-icon-building-fill, peut-être que ça serait bien d'uniformiser ici aussi

span>
span.fa.fa-phone>
br
span.fr-icon.fr-icon-phone-fill.fr-mr-1w[aria-hidden="true"]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

et peut-être que ça pourrait être utile d'avoir un petit helper qui permette de faire un truc dans ce genre :

Suggested change
span.fr-icon.fr-icon-phone-fill.fr-mr-1w[aria-hidden="true"]
= dsfr_icon("fr-icon-phone-fill", class: "fr-mr-1w")

et pour faciliter l'utilisation de la bonne icône, on pourrait même avoir des versions spécifiques de ce helper pour les icônes dont on a fixé la sémantique du genre

def dsfr_icon_for_motifs(class: nil)
  dsfr_icon("fr-icon-draft-fill", class:)
end

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes effectivement, j'y ai pensé et c’est dans ma todo
Je n'ai pas encore proposé d’implémentation parce que ça me semblait curieux qu’on soit toujours obligé de faire ça, mais je n’ai pas encore trouvé de meilleurs moyens nativement dans le DSFR
Je ferai certainement ça dans une PR prochainement

@AntoineGirard AntoineGirard force-pushed the agd/migrate-usager-rdv-icons-dsfr branch from 524a943 to 0839baa Compare January 6, 2025 15:07
@AntoineGirard AntoineGirard merged commit c837a5f into production Jan 6, 2025
15 checks passed
@AntoineGirard AntoineGirard deleted the agd/migrate-usager-rdv-icons-dsfr branch January 6, 2025 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

3 participants