Projet

Général

Profil

Development #41372

permettre une couleur différente des cases cochées personnalisées

Ajouté par Serghei Mihai il y a environ 4 ans. Mis à jour il y a environ 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
06 avril 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Pour avoir facilement des rendus comme dans la capture.


Fichiers

Révisions associées

Révision 1bb1d776 (diff)
Ajouté par Serghei Mihai il y a environ 4 ans

scss: define border and marker colors for custom radio/checkbox (#41372)

Historique

#2

Mis à jour par Serghei Mihai il y a environ 4 ans

#3

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

La description donnée dans le fichier d'aide ne me semble pas correspondre, être juste le copié/collé de widget-custom-radio-checkbox.

Et il y avait déjà une couleur et une possibilité de la paramétrer mais ça concernait aussi bien la bordure que le disque central, ici c'est pour donner une couleur particulière/différente au disque.

Je pense que du coup ça demande aussi adaptation à la description de $widget-custom-radio-checkbox-color, que la différence soit bien comprise.

Voire ça appelle à nommer les choses différemment, genre $widget-custom-radio-checkbox-marker-color, pour rester sur le même préfixe que l'autre (i.e. ça s'applique aussi bien aux cases à cocher qu'aux boutons radio) et pointer que c'est la couleur du marqueur qui est concernée.

#4

Mis à jour par Serghei Mihai il y a environ 4 ans

Frédéric Péters a écrit :

Je pense que du coup ça demande aussi adaptation à la description de $widget-custom-radio-checkbox-color, que la différence soit bien comprise.

Et comme la différence des 2 options maintenant sera qu'une définit la couleur de la bordure et l'autre du disque du widget, je pense que $widget-custom-radio-checkbox-color devrait plus logiquement s'appeler $widget-custom-radio-checkbox-border-color.

Et donc l'autre, comme tu suggères: $widget-custom-radio-checkbox-marker-color.

#5

Mis à jour par Serghei Mihai il y a environ 4 ans

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

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

$widget-custom-radio-checkbox-border-color non parce que si c'est la seule qui est mise elle concerne aussi le marqueur, sauf si l'idée est de changer ça, et demander désormais aux intégrations graphiques de poser deux variables là où elles en posaient une seule.

#7

Mis à jour par Serghei Mihai il y a environ 4 ans

Mon idée serait d'avoir:

$widget-custom-radio-checkbox-border-color: $button-background !default;
$widget-custom-radio-checkbox-marker-color: $widget-custom-radio-checkbox-border-color !default;

et dans les integrations remplacer $widget-custom-radio-checkbox-color par $widget-custom-radio-checkbox-border-color.

#8

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

J'ai bien compris, ton idée est $widget-custom-radio-checkbox-border-color détermine aussi la couleur du marqueur (sauf spécifié différente). Je ne trouve pas terriblement clair qu'une variable qui s'appelle "border" détermine la couleur du marqueur.

#9

Mis à jour par Serghei Mihai il y a environ 4 ans

Ok, autant garder la variable actuelle qui pilotera les couleurs des bordures et des marqueurs.

#10

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

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

Mis à jour par Serghei Mihai il y a environ 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
  • Assigné à mis à Serghei Mihai
commit 1bb1d77643d0596b6fd4f98d3444f570c67be44a (origin/master, origin/HEAD)
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Thu Apr 9 10:29:34 2020 +0200

    scss: define border and marker colors for custom radio/checkbox (#41372)
#12

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

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

Formats disponibles : Atom PDF