search

epfl-si / elements

Charte graphique Web EPFL
https://epfl-si.github.io/elements
22 stars 12 forks source link

Add switch #725

Closed williambelle closed 4 months ago

williambelle commented 4 months ago

WEBEVOL-246

Petits ajustements pour l'utilisation des switches.

Si vous pensez qu'il faut faire des changements plus important au niveau du design, vous pouvez modifier directement ma PR.

github-actions[bot] commented 4 months ago

Unit Test Results

    1 files      1 suites   0s :stopwatch: 278 tests 254 :heavy_check_mark: 0 :zzz:   0 :x: 24 :fire: 278 runs  230 :heavy_check_mark: 0 :zzz: 24 :x: 24 :fire:

For more details on these errors, see this check.

Results for commit 9463c7c1.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 4 months ago

🔎 Download the Backstop report for this pull request (link valid for 90 days):

alinekeller commented 4 months ago

@williambelle Le visuel des switch est très bien. Par contre j'aimerais améliorer l'indicateur de focus (outline) qui n'est pas du tout suffisant, ni sur les champs .custom-control ni sur les input de base. Deux questions:

williambelle commented 4 months ago

@alinekeller

  • Est-ce que je fais ça dans cette branche ou est-ce que j'en crée une séparée ?

Oui, tu peux le faire dans cette branche. J'ajouterai la mention co-authored-by.

  • Dans quel fichier CSS je mets les styles pour les .custom-control, qui concernent plusieurs composants ? Dans le premier de la liste (checkbox) ?

On pourrait rajouter un fichier dans assets/config/ si ça concerne plusieurs composants. Mais si il y a très peu de code, ça me convient dans checkbox.

alinekeller commented 4 months ago

@williambelle

On pourrait rajouter un fichier dans assets/config/ si ça concerne plusieurs composants. Mais si il y a très peu de code, ça me convient dans checkbox.

C'est 4 propriétés CSS, je vais le faire dans l'atom chechbox.