Projet

Général

Profil

Documentation #38320

$widget-focus-background n'est pas `white`, mais `inherit`

Ajouté par Thomas Jund (congés, retour le 29/04) il y a plus de 4 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Normal
Version cible:
-
Début:
09 décembre 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Erreur dans la doc scss


Fichiers

Révisions associées

Révision 817fb57a (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a plus de 4 ans

doc: change $widget-focus-background default value to $widget-background (#38320)

Historique

#1

Mis à jour par Thomas Jund (congés, retour le 29/04) 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

J'hésite là-dessus, je préférerais peut-être que la valeur par défaut soit réellement white, ou rester comme maintenant avec cette "erreur" dans la documentation, pour ne pas ajouter dans la documentation de propriété "inherit", dans l'idée que tout ça servira dans l'interface de contrôle de l'intégration graphique et de pas y exposer des valeurs qui demanderaient à comprendre les css.

(mais c'est comme tu veux, je suis neutre là-dessus).

#3

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 4 ans

À vrai dire je ne me suis poser la question. J'ai noté la différence et ouvert le ticket.

De mon point du vue, je n'imposerais pas de couleur de background en focus de champ. Je serais pour que le thème "épuré" (faudrait qu'on lui trouve un nom) soit le plus minimaliste possible.
Ni `white` et encore moins `inherit` qui n'a pas de sens. La valeur qui me semble la plus logique serait `$widget-background`

$widget-focus-background: $widget-background !default;
#4

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

Oui.

#5

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 4 ans

Proposition avec valeur $widget-background

#6

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

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

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 4 ans

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

    doc: change $widget-focus-background default value to $widget-background (#38320)

Formats disponibles : Atom PDF