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

[composant] Implémentation de sliderInput_dsfr() #102

Open
3 tasks done
ColinFay opened this issue Jun 30, 2023 · 6 comments
Open
3 tasks done

[composant] Implémentation de sliderInput_dsfr() #102

ColinFay opened this issue Jun 30, 2023 · 6 comments

Comments

@ColinFay
Copy link
Collaborator

ColinFay commented Jun 30, 2023

ETQ utilisateur/utilisatrice, je souhaite pouvoir ajouter un composant de type sliderInput_dsfr()` conforme au dsfr à mon application shiny dsfr

Critères d'acceptation des développements effectués

  • Une fonction implementée, documentée: `sliderInput_dsfr()' (en reprenant celle du sliderInput de shiny)

https://github.com/spyrales/shinygouv/blob/main/R/sliderinput_dsfr.R

image

  • Un exemple d'utilisation dans l'application de démonstration est disponible

https://ssm-ecologie.shinyapps.io/shinygouv-demo/#inputs_numeriques

-[] Avoir la même font que l'application pour les sliderInputs

Comment technique:

Pour ce qui est de la fonte, je vois deux pistes possibles:

  • Surcharger les classes de css shiny avec le css du dsfr
  • Recoder le sliderInput en entier
@ColinFay ColinFay self-assigned this Jun 30, 2023
@jdlom
Copy link
Contributor

jdlom commented Jun 30, 2023

@ColinFay
Copy link
Collaborator Author

ColinFay commented Jul 1, 2023

@jdlom il n'y a pas de slider dans le DSFR, j'ai donc repris le slider de shiny et modifié le css :) ça marche donc avec le slider par défaut, j'ai aussi code sliderInput_dsfr par homogénéité, mais les deux fonctionnent.

sliderInput_dsfr <- shiny::sliderInput
et https://github.com/spyrales/shinygouv/blob/main/inst/external_deps/slider.css

@jdlom
Copy link
Contributor

jdlom commented Jul 1, 2023

@ColinFay j'ai pas été clair , le slider c'est une bonne idée, c'est juste qu'il y'en a deux sur l'App de démo

@Cervangirard
Copy link
Contributor

Ceci a été supprimé

@MurielleDelmotte MurielleDelmotte changed the title feat: sliderInput [composant] Implémentation de sliderInput_dsfr() Jul 6, 2023
@jengelaere
Copy link
Contributor

y a 'il moyen de mettre la bonne police pour les chiffres du slider ?
image

@jengelaere jengelaere removed their assignment Jul 19, 2023
@Cervangirard Cervangirard assigned jdlom and unassigned jdlom Aug 2, 2023
@Cervangirard
Copy link
Contributor

Comme expliquer dans le comment techniques du ticket, il y a deux options. je reste à votre dispo pour review une MR au besoin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

5 participants