Projet

Général

Profil

Development #23533

scss: possibilité d'avoir les étapes d'un formulaire sous forme de cercles

Ajouté par Serghei Mihai il y a presque 6 ans. Mis à jour il y a environ 5 ans.

Statut:
Rejeté
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
02 mai 2018
Echéance:
20 juillet 2018
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Dans quelques thèmes récents les étapes d'un formulaire sont designés sous forme de cercles avec une bordure avec la mise en exergue de l'étape active.

On pourrait généraliser cela afin de pouvoir le faire à coup de variables au lieu de mettre plein de code dans le _custom.scss.


Fichiers

amiens.png (41,3 ko) amiens.png Serghei Mihai, 02 mai 2018 13:43
strasbourg.png (619 ko) strasbourg.png Serghei Mihai, 02 mai 2018 13:43
_circle-steps.scss (1,56 ko) _circle-steps.scss Anonyme, 16 mai 2018 14:51
Screenshot-2018-6-11 toodego - Démarches - Voirie Signalisation.png (6,33 ko) Screenshot-2018-6-11 toodego - Démarches - Voirie Signalisation.png Frédéric Péters, 11 juin 2018 16:04
_circle-steps.scss (3,02 ko) _circle-steps.scss Anonyme, 26 juin 2018 16:06
0001-misc-add-circle-steps-option-in-wcs-23533.patch (8,78 ko) 0001-misc-add-circle-steps-option-in-wcs-23533.patch Anonyme, 18 juillet 2018 14:09
demarches-wcs.dev-eshowk.ddns.entrouvert.org_formation_test-steps_.png (71,7 ko) demarches-wcs.dev-eshowk.ddns.entrouvert.org_formation_test-steps_.png Anonyme, 18 juillet 2018 14:09
demarches-wcs.dev-eshowk.ddns.entrouvert.org_formation_test-steps_(Galaxy S5).png (159 ko) demarches-wcs.dev-eshowk.ddns.entrouvert.org_formation_test-steps_(Galaxy S5).png Anonyme, 18 juillet 2018 14:09
_vars.scss (1,33 ko) _vars.scss Anonyme, 18 juillet 2018 14:09
demarches-wcs.dev-eshowk.ddns.entrouvert.org_formation_test-steps_ (1).png (50,8 ko) demarches-wcs.dev-eshowk.ddns.entrouvert.org_formation_test-steps_ (1).png Anonyme, 23 juillet 2018 16:32
demarches-wcs.dev-eshowk.ddns.entrouvert.org_formation_test-steps_ (2).png (95,1 ko) demarches-wcs.dev-eshowk.ddns.entrouvert.org_formation_test-steps_ (2).png Anonyme, 23 juillet 2018 16:32
demarches-wcs.dev-eshowk.ddns.entrouvert.org_formation_test-steps_.png (97,7 ko) demarches-wcs.dev-eshowk.ddns.entrouvert.org_formation_test-steps_.png Anonyme, 23 juillet 2018 16:32
0002-rouen-activate-circle-steps-25104.patch (588 octets) 0002-rouen-activate-circle-steps-25104.patch Anonyme, 23 juillet 2018 16:32
0001-add-option-to-display-circle-steps-23533.patch (9,39 ko) 0001-add-option-to-display-circle-steps-23533.patch Anonyme, 23 juillet 2018 16:32
0001-add-option-to-display-circle-steps-23533.patch (9,36 ko) 0001-add-option-to-display-circle-steps-23533.patch Anonyme, 23 juillet 2018 17:28
0002-rouen-activate-circle-steps-25104.patch (588 octets) 0002-rouen-activate-circle-steps-25104.patch Anonyme, 23 juillet 2018 17:28
demarches-wcs.dev-eshowk.ddns.entrouvert.org_formation_test-steps_ (3).png (87,4 ko) demarches-wcs.dev-eshowk.ddns.entrouvert.org_formation_test-steps_ (3).png Anonyme, 23 juillet 2018 17:29
0001-add-option-to-display-circle-steps-23533.patch (9,36 ko) 0001-add-option-to-display-circle-steps-23533.patch Anonyme, 23 juillet 2018 18:00
0002-rouen-activate-circle-steps-25104.patch (588 octets) 0002-rouen-activate-circle-steps-25104.patch Anonyme, 23 juillet 2018 18:00
0001-adjustments.patch (2,1 ko) 0001-adjustments.patch Frédéric Péters, 26 juillet 2018 14:03
0002-rouen-activate-circle-steps-25104.patch (588 octets) 0002-rouen-activate-circle-steps-25104.patch Anonyme, 26 juillet 2018 15:18
0001-add-option-to-display-circle-steps-23533.patch (8,87 ko) 0001-add-option-to-display-circle-steps-23533.patch Anonyme, 26 juillet 2018 15:18
clapotis-mobile-steps.png (6,53 ko) clapotis-mobile-steps.png Frédéric Péters, 26 juillet 2018 15:24
0001-add-option-to-display-circle-steps-23533.patch (8,87 ko) 0001-add-option-to-display-circle-steps-23533.patch Anonyme, 27 juillet 2018 09:24
demarches-wcs.dev-eshowk.ddns.entrouvert.org_test-steps_.png (25,2 ko) demarches-wcs.dev-eshowk.ddns.entrouvert.org_test-steps_.png Anonyme, 27 juillet 2018 09:24
demarches-wcs.dev-eshowk.ddns.entrouvert.org_test-steps_ (1).png (57,2 ko) demarches-wcs.dev-eshowk.ddns.entrouvert.org_test-steps_ (1).png Anonyme, 27 juillet 2018 09:24
demarches-wcs.dev-eshowk.ddns.entrouvert.org_test-steps_ (2).png (61,4 ko) demarches-wcs.dev-eshowk.ddns.entrouvert.org_test-steps_ (2).png Anonyme, 27 juillet 2018 10:08
0002-rouen-activate-circle-steps-25104.patch (588 octets) 0002-rouen-activate-circle-steps-25104.patch Anonyme, 27 juillet 2018 10:08
0001-add-option-to-display-circle-steps-23533.patch (8,82 ko) 0001-add-option-to-display-circle-steps-23533.patch Anonyme, 27 juillet 2018 10:08
demarches-wcs.dev-eshowk.ddns.entrouvert.org_test-steps_ (5).png (70,4 ko) demarches-wcs.dev-eshowk.ddns.entrouvert.org_test-steps_ (5).png Anonyme, 27 juillet 2018 11:16
demarches-wcs.dev-eshowk.ddns.entrouvert.org_test-steps_(Galaxy S5).png (159 ko) demarches-wcs.dev-eshowk.ddns.entrouvert.org_test-steps_(Galaxy S5).png Anonyme, 27 juillet 2018 11:16
demarches-wcs.dev-eshowk.ddns.entrouvert.org_test-steps_ (3).png (101 ko) demarches-wcs.dev-eshowk.ddns.entrouvert.org_test-steps_ (3).png Anonyme, 27 juillet 2018 11:16
0001-add-variable-for-native-circle-steps-23533.patch (9,15 ko) 0001-add-variable-for-native-circle-steps-23533.patch Anonyme, 27 juillet 2018 11:16
armor.png (8,49 ko) armor.png Frédéric Péters, 27 juillet 2018 11:25
0001-add-variable-for-native-circle-steps-23533.patch (9,18 ko) 0001-add-variable-for-native-circle-steps-23533.patch Anonyme, 27 juillet 2018 18:55
un-demarches-wcs.dev-eshowk.ddns.entrouvert.org_test-steps_ (2).png (81,9 ko) un-demarches-wcs.dev-eshowk.ddns.entrouvert.org_test-steps_ (2).png Anonyme, 27 juillet 2018 19:07
demarches-wcs.dev-eshowk.ddns.entrouvert.org_test-steps_ (9).png (37,7 ko) demarches-wcs.dev-eshowk.ddns.entrouvert.org_test-steps_ (9).png Anonyme, 30 juillet 2018 12:27
Capture d’écran de 2018-07-30 12-18-45.png (10,9 ko) Capture d’écran de 2018-07-30 12-18-45.png Anonyme, 30 juillet 2018 12:27
0001-add-variable-for-native-circle-steps-23533.patch (9,15 ko) 0001-add-variable-for-native-circle-steps-23533.patch Anonyme, 30 juillet 2018 12:29
Capture d’écran de 2018-07-30 12-18-45.png (10,9 ko) Capture d’écran de 2018-07-30 12-18-45.png Anonyme, 30 juillet 2018 19:33
screenshot.png (9,25 ko) screenshot.png captures zoom firefox et chromium Frédéric Péters, 30 juillet 2018 21:27
0001-add-variable-for-native-circle-steps-23533.patch (9,18 ko) 0001-add-variable-for-native-circle-steps-23533.patch Anonyme, 16 août 2018 15:56
0001-add-variable-for-native-circle-steps-23533.patch (9,18 ko) 0001-add-variable-for-native-circle-steps-23533.patch Anonyme, 29 août 2018 10:09

Demandes liées

Lié à Intégrations graphiques Publik - Development #25513: hautes-alpes-2018: basculer sur les nouvelles étapes en cerclesRejeté27 juillet 2018

Actions

Historique

#1

Mis à jour par Frédéric Péters il y a presque 6 ans

Tu aurais des captures de ces intégrations ?

#2

Mis à jour par Serghei Mihai il y a presque 6 ans

Amiens et Strasbourg.

#3

Mis à jour par Anonyme il y a presque 6 ans

j'ai inclus un fichier _circle-steps.scss dans le thème hautes-alpes-2018 qui peut servir de base de travail à ce ticket

#4

Mis à jour par Anonyme il y a presque 6 ans

  • Assigné à mis à Anonyme
#6

Mis à jour par Anonyme il y a presque 6 ans

Je proposerai une version après la validation du ticket lié #24259 où je sortirai et simplifierai encore _circle-steps.scss

#8

Mis à jour par Anonyme il y a presque 6 ans

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

Perso je préfère les petits cercles comme sur https://demarches.guichet-recette.grandlyon.com/signalements/voirie-et-signalisation/ (en pièce jointe).

D'accord, donc ajouter la possibilité de spécifier un diamètre en variable ?

#9

Mis à jour par Frédéric Péters il y a presque 6 ans

D'accord, donc ajouter la possibilité de spécifier un diamètre en variable ?

Il y a deux captures, une Amiens en mode vertical, une Strasbourg en mode horizontal, j'en ai ajouté une de GNM, en mode vertical répondant à "sous forme de cercles". Je ne dis pas qu'il faut du code gérant toutes les possibilités.

Bien sûr je préférerais un rendu que je trouve agréable, mais il faut mesurer les goûts des autres et ce que tu peux faire côté code.

#10

Mis à jour par Anonyme il y a presque 6 ans

Une version avec variables extraite d'amiens-metropole

#11

Mis à jour par Serghei Mihai il y a presque 6 ans

#12

Mis à jour par Anonyme il y a presque 6 ans

  • Echéance mis à 20 juillet 2018
  • échéance posée pour essayer d'utiliser ça à rouen
#13

Mis à jour par Anonyme il y a presque 6 ans

Voilà $circle-steps: true; et 14 variables (avec une valeur défaut).

  • appliqué à rouen, avec le fichier _vars.scss en PJ.
  • les captures aussi pour vérifier que c'est ce qui est attendu.
#14

Mis à jour par Frédéric Péters il y a presque 6 ans

J'ai l'impression qu'il y a eu réindentation qui casse toute lisibilité au patch.

#15

Mis à jour par Frédéric Péters il y a presque 6 ans

Pour les auteurs de thème les variables doivent être documentées, cf help/fr/misc-scss.page, ça va peut-être donner l'idée qu'il y en a trop.

Je dirais qu'on pourrait juste avoir diamètre et couleurs fond/texte des pastilles.

Le raisonnement général à porter c'est qu'au-delà de quelques-unes vraiment signifiantes, s'il y a volonté dans une intégration graphique de changer un aspect, par exemple mettre en majuscules le nom des étapes, il y sera plus rapide dans un _custom.scss de taper div#steps span.label { text-transform: uppercase }; que de chercher si ça existe sous forme de paramétrage.

$circle-steps-marker-diameter: 4vw !default;

Je ne pense pas que faire varier le diamètre selon la largeur de l'écran soit une bon paramétrage par défaut. Dans la pratique les intégrations fixent pour la plupart une largeur de page, à partir de laquelle le comportement est d'augmenter les marges latérales et c'est tout, bizarre alors d'y avoir les pastilles continuer à changer de taille.

@import 'flexbox_mixins';

Ok pour que ça soit utilisé dans des intégrations graphiques particulières mais je ne l'aime toujours pas et préfère les choses gérées directement. Et pour ce qui ne sera interprété que côté mobile, on peut vraiment sans aucun artifice utiliser exclusivement les vraies propriétés.

~~

Ce n'est pas aligné.

~~

Et bien sûr tout à fait personnel mais je ne trouve pas ça graphiquement réussi ou propre.

#16

Mis à jour par Anonyme il y a presque 6 ans

Pour l'indentation, il en manquait une dans _wcs.scss à la ligne de @if ($responsive-steps == horizontal) {.
Je dois l'enlever.

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

Pour les auteurs de thème les variables doivent être documentées, cf help/fr/misc-scss.page, ça va peut-être donner l'idée qu'il y en a trop.

OK

Je dirais qu'on pourrait juste avoir diamètre et couleurs fond/texte des pastilles.

OK

Le raisonnement général à porter c'est qu'au-delà de quelques-unes vraiment signifiantes, s'il y a volonté dans une intégration graphique de changer un aspect, par exemple mettre en majuscules le nom des étapes, il y sera plus rapide dans un _custom.scss de taper div#steps span.label { text-transform: uppercase }; que de chercher si ça existe sous forme de paramétrage.

$circle-steps-marker-diameter: 4vw !default;

Je ne pense pas que faire varier le diamètre selon la largeur de l'écran soit une bon paramétrage par défaut. Dans la pratique les intégrations fixent pour la plupart une largeur de page, à partir de laquelle le comportement est d'augmenter les marges latérales et c'est tout, bizarre alors d'y avoir les pastilles continuer à changer de taille.

OK

@import 'flexbox_mixins';

Ok pour que ça soit utilisé dans des intégrations graphiques particulières mais je ne l'aime toujours pas et préfère les choses gérées directement. Et pour ce qui ne sera interprété que côté mobile, on peut vraiment sans aucun artifice utiliser exclusivement les vraies propriétés.

C'est répétitif et dommage se passer de l'outillage Sass à mon avis, mais OK.

~~

Ce n'est pas aligné.

Oui je remarque les traits horizontaux. Quoi d'autre sinon ?

~~

Et bien sûr tout à fait personnel mais je ne trouve pas ça graphiquement réussi ou propre.

Et donc ça ne sera pas accepté parce que c'est pas graphiquement réussi ou propre, ou bien je peux avancer quand même.

#17

Mis à jour par Frédéric Péters il y a presque 6 ans

Et bien sûr tout à fait personnel mais je ne trouve pas ça graphiquement réussi ou propre.

Et donc ça ne sera pas accepté parce que c'est pas graphiquement réussi ou propre, ou bien je peux avancer quand même.

Je ne mettrai pas d'objection là-dessus si des gens trouvent ça réussi.

#18

Mis à jour par Emmanuel Cazenave il y a presque 6 ans

Je ne suis pas très fan du rendu des barres de liaison :

  • en mode horizontal, elles ne touchent pas la rond de gauche et sont un peu trop basses
  • en mode vertical, leur interruption rend le truc vraiment bizarre je trouve (peut être pas d'interruption avec le texte qui passe par dessus ?)

Mais en soit je trouve ça bien qu'il y ait des barres (et idem pour Brice).

#19

Mis à jour par Anonyme il y a plus de 5 ans

Emmanuel Cazenave a écrit :

Je ne suis pas très fan du rendu des barres de liaison :

  • en mode horizontal, elles ne touchent pas la rond de gauche et sont un peu trop basses
  • en mode vertical, leur interruption rend le truc vraiment bizarre je trouve (peut être pas d'interruption avec le texte qui passe par dessus ?)

Voilà une seconde mouture.

  • J'ai trouvé un code pour que les traits touchent les cercles, mais à condition que le texte ne dépasse pas 5 lignes (voir la 3e capture qui le montre)
  • J'ai ajouté la documentation, il ne reste que 5 variables de paramétrage, donc une pour afficher ou non les lignes entre les cercles.
Pour ça, j'ai dû faire quelques adaptations:
  • basculé les labels à côté des numéros pour ne pas avoir d'éléments qui se chevauchent.
  • donné plus de largeur aux étapes parce que le précédent réglage serrait trop les labels.
#20

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

Ça ne fait pas curieux d'avoir le texte centré ? (moi je trouve)

height: calc(#{$_steps-height} + #{$circle-steps-diameter}/2 + #{$_steps-border});

Ici et ailleurs, le calcul me semble pouvoir être assuré totalement côté sass.

#21

Mis à jour par Anonyme il y a plus de 5 ans

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

Ça ne fait pas curieux d'avoir le texte centré ? (moi je trouve)

Si

Ici et ailleurs, le calcul me semble pouvoir être assuré totalement côté sass.

Ok

#22

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

Comme les cercles/disques ont une largeur paire (56px), la ligne qui fait un pixel n'apparait pas centrée.

#23

Mis à jour par Anonyme il y a plus de 5 ans

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

Comme les cercles/disques ont une largeur paire (56px), la ligne qui fait un pixel n'apparait pas centrée.

OK, bien vu !

#24

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

      error ../includes/_form-steps.scss (Line 208: Undefined variable: "$responsive-steps".)

Parce que $responsive-steps est défini dans _wcs.scss après l'import de _form-steps.scss.

#25

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

Testé dans clapotis-les-canards : l'étape courante est affichée en blanc sur fond blanc, les étapes avec des libellés un peu long prennent trop de largeur et poussent le contenu de la démarche dessous.

#26

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

les étapes avec des libellés un peu long prennent trop de largeur et poussent le contenu de la démarche dessous

En gros la modification au _wcs.scss doit être retirée. Si on veut contrôler la largeur de sa barre latérale, il faut autre chose.

Retirer aussi le padding ajouté à div#steps (padding: 0 1em), que les étapes prennent la largeur de la barre latérale, et pas largeur - 2em.

#27

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

(patch avec ces modifs, + modif pour ne pas jouer avec de la marge négative).

#28

Mis à jour par Anonyme il y a plus de 5 ans

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

(patch avec ces modifs, + modif pour ne pas jouer avec de la marge négative).

Merci.

#29

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

En mobile (clapotis-les-canards) les ronds sont trop larges et deviennent des ellipses. Et la hauteur est paire et du coup nécessairement pas aligné (même si ça me choque moins ici) Aussi toutes les étapes centrées mais du coup on peut ne pas voir l'étape courante. (à Strasbourg sur une apparence similaire l'espace entre les blocs s'ajuste selon la largeur et on arrive à en faire tenir 6). (on peut aussi se dire que de tels formulaires multi-pages n'ont pas vocation à être saisis via mobile).

#30

Mis à jour par Anonyme il y a plus de 5 ans

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

En mobile (clapotis-les-canards) les ronds sont trop larges et deviennent des ellipses. Et la hauteur est paire et du coup nécessairement pas aligné (même si ça me choque moins ici)

Corrigé, mais à noter qu'en responsive au-delà de 10 étapes, les ronds ne sont plus des cercles.

Aussi toutes les étapes centrées mais du coup on peut ne pas voir l'étape courante. (à Strasbourg sur une apparence similaire l'espace entre les blocs s'ajuste selon la largeur et on arrive à en faire tenir 6). (on peut aussi se dire que de tels formulaires multi-pages n'ont pas vocation à être saisis via mobile).

J'ai choisi de laissé centré, mais avec un flex-wrap actif.

#31

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

l'étape courante est affichée en blanc sur fond blanc,

c'est toujours le cas.

#33

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

Pour assurer que ça soit utile, tu modifierais hautes-alpes-2018 pour passer par ce fichier ? Et sans doute aussi vérifier le rendu dans l'une ou l'autre intégration native.

#34

Mis à jour par Anonyme il y a plus de 5 ans

  • Lié à Development #25513: hautes-alpes-2018: basculer sur les nouvelles étapes en cercles ajouté
#35

Mis à jour par Anonyme il y a plus de 5 ans

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

Pour assurer que ça soit utile, tu modifierais hautes-alpes-2018 pour passer par ce fichier ? Et sans doute aussi vérifier le rendu dans l'une ou l'autre intégration native.

OK pour cotes-darmor, et hautes-alpes-2018 dans le ticket lié.
J'ai fait quelques modifications pour que l'option "sans-traits" permette de revenir à des labels centrés sous les numéros, comme c'est le cas par pour le thèmes des H-A.
Nouveau message de commit pour plus de clarté.

#36

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

Côtes d'Armor avec les traits on voit que ce n'est pas centré, cf zoom

#37

Mis à jour par Anonyme il y a plus de 5 ans

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

Côtes d'Armor avec les traits on voit que ce n'est pas centré, cf zoom

un right: calc(50% - #{$la-bordure-du-cercle}); pour centrer au mieux, mais avec un diametre impair pour éviter les problèmes d'alignement précédents, on arrive sur un autre problème de demi-pixels, et ça peut placer le trait de 1px à cheval sur la grille de l'écran, et donc être affiché en 2px effectifs. J'assume cette "fausse" épaisseur, et on fixe un minimum de 2px réels.
Je pense qu'on ne pourra pas optimiser plus ce rendu.

#38

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

(...) un diametre impair (...) on fixe un minimum de 2px réels.

Mais diamètre impair et trait de 2 pixels, c'est garantie de ne jamais être centré.

Je pense qu'on ne pourra pas optimiser plus ce rendu.

Heureusement il existe dans sass floor(), ceil() et round() pour te contredire.

#39

Mis à jour par Anonyme il y a plus de 5 ans

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

(...) un diametre impair (...) on fixe un minimum de 2px réels.

Mais diamètre impair et trait de 2 pixels, c'est garantie de ne jamais être centré.

Je pense qu'on ne pourra pas optimiser plus ce rendu.

Heureusement il existe dans sass floor(), ceil() et round() pour te contredire.

Oui mais on ne peut pas faire floor(calc(...))
J'ai essayé de nombreuses combinaisons, et je n'ai rien trouvé de plus satisfaisant que ce dernier patch.
(L'affichage se fait parfois à cheval sur deux pixels, c'est ce que je voulais dire avant, même 1px de largeur peut donner un affichage sur 2 pixels réels)

Je suis allé sur un diamètre pair, et un positionnement du trait avec left:50%; plutôt qu'un calcul compliqué avec right. C'est le meilleur rendu pour ma part.

J'en ai profité pour un floor() sur la longueur du trait horizontal en responsive, pour éviter les demi-pixels.

#40

Mis à jour par Anonyme il y a plus de 5 ans

Elias Showk a écrit :

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

(...) un diametre impair (...) on fixe un minimum de 2px réels.

Mais diamètre impair et trait de 2 pixels, c'est garantie de ne jamais être centré.

Je pense qu'on ne pourra pas optimiser plus ce rendu.

Heureusement il existe dans sass floor(), ceil() et round() pour te contredire.

Oui mais on ne peut pas faire floor(calc(...))
J'ai essayé de nombreuses combinaisons, et je n'ai rien trouvé de plus satisfaisant que ce dernier patch.
(L'affichage se fait parfois à cheval sur deux pixels, c'est ce que je voulais dire avant, même 1px de largeur peut donner un affichage sur 2 pixels réels)

Je suis allé sur un diamètre pair, et un positionnement du trait avec left:50%; plutôt qu'un calcul compliqué avec right. C'est le meilleur rendu pour ma part.

J'en ai profité pour un floor() sur la longueur du trait horizontal en responsive, pour éviter les demi-pixels.

#41

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

Oui mais on ne peut pas faire floor(calc(...))

Mais il n'y a pas de calc() dans le dernier patch. (???)

#42

Mis à jour par Anonyme il y a plus de 5 ans

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

Oui mais on ne peut pas faire floor(calc(...))

Mais il n'y a pas de calc() dans le dernier patch. (???)

J'en ai plus besoin, je me sers d'un left: 50%;

#43

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

Mais l'argumentation sur le calc() qui empêchait d'avoir quelque chose de correct ? Je ne sais pas trop ce qui reste vrai ou pas de ton dernier commentaire. (et reste persuadé qu'on peut aligner correctement une ligne d'1px).

#44

Mis à jour par Anonyme il y a plus de 5 ans

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

Mais l'argumentation sur le calc() qui empêchait d'avoir quelque chose de correct ? Je ne sais pas trop ce qui reste vrai ou pas de ton dernier commentaire. (et reste persuadé qu'on peut aligner correctement une ligne d'1px).

Je trouve inutile d'utiliser le calc pour résoudre le prob du centrage qu'il ne résout finalement pas. Je reviens vers left: 50% pour ne pas rendre le truc compliqué pour rien.

Je trouve les dernières captures correctes. Je joins une copie une nouvelle fois, et je m'arrête là pour le moment car j'ai essayé bcp d'options sans succès, celle là est la plus satisfaisant.

Quand je trouverai une optimisation, j'en ferait un patch, mais si d'ici là il faut passer le patch en mon absence, servez-vous.

#45

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

--- a/static/includes/_form-steps.scss
+++ b/static/includes/_form-steps.scss
@@ -1,6 +1,6 @@
 $circle-steps: false !default;
 $circle-steps-with-lines: true !default;
-$circle-steps-diameter: 56px !default;
+$circle-steps-diameter: 55px !default;
 $circle-steps-number-color: #868686 !default;
 $circle-steps-current-number-color: white !default;
 $circle-steps-current-marker-background: $primary-color !default;
@@ -105,7 +105,7 @@ $_steps-border: 1px !default;
                                        height: $_steps-height + $circle-steps-diameter - $_steps-border;
                                        width: $_circle-lines-thickness;
                                        bottom: $circle-steps-diameter - $_steps-border;
-                                       left: 50%;
+                                       left: floor($circle-steps-diameter / 2) - 1px;
                                        background-color: $circle-steps-number-color;
                                        padding: 0;
                                }
#46

Mis à jour par Anonyme il y a plus de 5 ans

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

[...]

C'est corrigé

#47

Mis à jour par Anonyme il y a plus de 5 ans

#48

Mis à jour par Anonyme il y a plus de 5 ans

#49

Mis à jour par Anonyme il y a plus de 5 ans

Aucun changement, juste pour remettre le dernier patch en haut...

#50

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

En fait je me trouverais triste à accepter ce patch, que je n'aime pas du tout, dès le fond, qui sectionne en deux options totalement différentes le rendu, avec des règles CSS sans rapport entre elles, plutôt que construire sur de l'existant (option pour que le numéro de l'étape soit sur la même ligne, option pour le fond (couleur/forme) de l'étape, option pour ajouter un trait, etc.).

Du coup, si les échéances ne rendent pas ça possible, qu'on continue à copier un circle-steps.scss dans les intégrations graphiques qui le souhaiterait, sur lesquelles je fermerai les yeux.

#51

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

#52

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

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

Je pense que la possibilité reste pertinente mais ce ticket est devenu trop long pour être utile.

Formats disponibles : Atom PDF