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

PGF-732: disabled Button and DaButton without hover style (fix) #142

Merged
merged 2 commits into from May 31, 2021

Conversation

OliviaGometz
Copy link
Contributor

Ce qui a été fait

Fix sur les composants Button et DaButton : désormais, lorsqu'ils sont disabled, ils n'ont aucun effet au hover.

Explication du bug

Le bug provenait de la structure du CSS (avec styled-components, il est vite périlleux de faire du style conditionnel lorsque celui-ci touche à un composant dont le style est visible lorsqu'il est contenu à l'intérieur d'une autre balise HTML). Exemple de CSS sujet à bugs :

    button:hover &,
    button:active &,
    button:focus &,
    a:hover &,
    a:active &,
    a:focus & {
        &::before {
            transform: scale(1);
        }
    }

Seul, ce bout de CSS fonctionne bien, mais s'il est contenu dans une variable appelée de manière conditionnelle (exemple : ${props => (props.isDisabled ? disabledStyle : null)};), il "persiste" même si la condition n'est pas remplie (cela tient à la manière dont les classes CSS sont générées).

Comment tester ?

  • Aller sur la story du Button, tester le hover, puis passer en disabled et re-tester le hover : il ne doit plus y avoir le moindre effet. Retirez le disabled et vérifier que le hover est à nouveau visible.
  • Faire exactement le même test sur la story du DaButton.

src/lib/Button/style/base.js Outdated Show resolved Hide resolved
@OliviaGometz OliviaGometz merged commit bd9ce81 into master May 31, 2021
@OliviaGometz OliviaGometz deleted the pgf-732-fix-buttons-disabled-style branch May 31, 2021 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants