Projet

Général

Profil

Bug #38567

button:disabled, passer la bordure en niveau de gris

Ajouté par Thomas Jund il y a plus de 4 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Thomas Jund
Version cible:
-
Début:
16 décembre 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Il est possible de paramétrer les couleurs du bouton à travers

$button-background
$button-color
$button-border

et dans sa version :disabled les couleurs du background et du texte sont basculées en niveau de gris.

background: grayscale($button-background);
color: grayscale($button-color);

Mais pas la couleur de la bordure, ce qui est assez étrange.
Je propose de basculer également la couleur de la bordure en niveau de gris.

Fichiers

0001-scss-toggle-button-border-in-greyscale-when-he-s-dis.patch (714 octets) 0001-scss-toggle-button-border-in-greyscale-when-he-s-dis.patch Thomas Jund, 16 décembre 2019 17:02
0001-scss-try-button-border-color-before-grayscale-it-385.patch (890 octets) 0001-scss-try-button-border-color-before-grayscale-it-385.patch Thomas Jund, 17 décembre 2019 14:41

Révisions associées

Révision 10d2a6ba (diff)
Ajouté par Thomas Jund il y a plus de 4 ans

scss: toggle button border in greyscale when he's :disabled (#38567)

Révision 3947aad0 (diff)
Ajouté par Frédéric Péters il y a plus de 4 ans

scss: use transparent as undefined value for extract-color() (#38567)

Historique

#1

Mis à jour par Thomas Jund il y a plus de 4 ans

#2

Mis à jour par Frédéric Péters il y a plus de 4 ans

  • Statut changé de Solution proposée à Solution validée

Yes, c'était sans doute pas fait parce qu'on n'avait pas extract-color avant.

#3

Mis à jour par Thomas Jund il y a plus de 4 ans

  • Statut changé de Solution validée à Solution déployée
commit 10d2a6ba539931a902d67c8a6fc0689eba1b8784 (HEAD -> master, origin/master, origin/HEAD)
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Mon Dec 16 17:01:16 2019 +0100

    scss: toggle button border in greyscale when he's :disabled (#38567)
#4

Mis à jour par Frédéric Péters il y a plus de 4 ans

  • Statut changé de Solution déployée à En cours

fail sur

sassc --sourcemap static/hautes-alpes-2018/style.scss static/hautes-alpes-2018/style.css
Error: argument `$color` of `grayscale($color)` must be a color

       Backtrace:
           static/includes/_forms.scss:108, in function `grayscale`
           static/includes/_forms.scss:108
        on line 108 of static/includes/_forms.scss
>>         border-color: grayscale(extract-color($button-border));
   ----------------^
#5

Mis à jour par Thomas Jund il y a plus de 4 ans

  • Statut changé de En cours à Solution proposée
#7

Mis à jour par Frédéric Péters il y a plus de 4 ans

Après avoir testé que ça marche, je serais plutôt pour :

 @function extract-color($border){
        @each $part in $border { @if type-of($part) == color { @return $part; } }
-       @return null;
+       @return transparent;
 }

Histoire de ne pas devoir encadrer tous les les appels à extract-color().

#8

Mis à jour par Thomas Jund il y a plus de 4 ans

Bien vu, ce qui me fait pensé que ça va encore coincer sur les keywords inherit (pas très utile), currentColor (parfois utile) et plus tard sur initial et unset.
currentColor étant la valeur par défaut.

#9

Mis à jour par Frédéric Péters il y a plus de 4 ans

  • Statut changé de Solution proposée à Résolu (à déployer)

Histoire de réparer le build, je prends "Bien vu" comme une validation et j'ai envoyé ça.

commit 3947aad0260eafb01effbb66a59db8e6b36aa2e0
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Dec 18 11:10:01 2019 +0100

    scss: use transparent as undefined value for extract-color() (#38567)
#11

Mis à jour par Thomas Jund il y a plus de 4 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF