pix-ui
pix-ui copied to clipboard
1024pix
[BUGFIX] Permet le clique entre les input `radio` ou `checkbox` et leur label
:christmas_tree: Problème
Depuis la v45, on ne peut plus cliquer entre un input "radio" ou "checkbox" et son label associé.
:gift: Proposition
Replacer l'input à l'intérieur du label et utiliser flexbox comme avant.
:star2: Remarques
R AS
:santa: Pour tester
Vérifier qu'on peut correctement interagir avec les composants PixRadioButton et PixCheckbox.
Une fois l'application déployée, elle sera accessible à cette adresse https://ui-pr607.review.pix.fr Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-ui-review-pr607/environment
Est ce que l'ajout d'un yield input dans PixLabel serait plus "clean" . ( if has-block input ) etc... ? et gérer ça côté PixLabel ?
Est ce que l'ajout d'un yield input dans PixLabel serait plus "clean" . ( if has-block input ) etc... ? et gérer ça côté PixLabel ?
J'aime bien la simplicité de l'interface actuelle, j'ai peur qu'ajouter un yield ajoute une variante d'usage qu'on devra maintenir derrière.
J'aime bien la simplicité de l'interface actuelle, j'ai peur qu'ajouter un
yieldajoute une variante d'usage qu'on devra maintenir derrière.
Je comprends les deux points de vue. De mon côté même si j'ai du mal à voir les impacts (négatifs) possibles de cette PR, ça me fait un peu peur de voir le composant radio-button intégrer des usages dans le composant label sans que ce dernier n'ait été pensé pour et testé en conséquence
De mon côté même si j'ai du mal à voir les impacts (négatifs) possibles de cette PR
Pour moi l'impact principal c'est le lien fort entre le composant PixRadioButton et la classe pix-label qui est interne à PixLabel. Si un jour, le nom pix-label évolue, les styles risquent de casser. Mais :
- je ne vois pas pourquoi ce nom évoluerait,
- j'espère qu'on s'en rendrait compte 😬
ajouter un attribut au pixLabel @wrappedElement true|false ? qui ajoute un modifier à la classe pixLabel ?
:tada: This PR is included in version 45.4.1 :tada:
The release is available on:
Your semantic-release bot :package::rocket:
