Project

General

Profile

Bug #41387

champs civilité sur la page de finalisation de création de compte

Added by Frédéric Péters 8 months ago. Updated 4 months ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
06 Apr 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Ils s'affichent avec un <ul> natif avec madame / monsieur sur deux lignes et des boulettes devant.

0001-scss-style-civility-field-on-registration-page-41387.patch View (904 Bytes) Frédéric Péters, 06 Apr 2020 06:19 PM

Associated revisions

Revision 7d3ade95 (diff)
Added by Frédéric Péters 8 months ago

scss: style civility field on registration page (#41387)

Revision fa69cfd8 (diff)
Added by Frédéric Péters 4 months ago

scss: style civility field on registration page (#41387)

History

#1 Updated by Frédéric Péters 8 months ago

#2 Updated by Thomas Jund 8 months ago

On pourrait commencer à ne plus mettre d'informations DOM dans les nouveaux selecteurs ?

#3 Updated by Frédéric Péters 8 months ago

Là on se cale sur un existant et je préfère vraiment ne pas un jour à avoir à me poser la question de pourquoi le sélecteur pour le champ sur la page de création de compte a moins de spécificité que sur la page d'édition de compte.

#4 Updated by Thomas Jund 8 months ago

Le CSS cherche à compenser les lacunes du markup HTML :

Une balise p est utilisé comme `widget wrapper`. Mais la balise p n'est pas autorisé à posséder des balises ul. Les ul sont éjectés de la balise par le navigateur. Le markup casse et on est ensuite obligé de compenser le rythme vertical dégradé à coup de margin-negatif.

Bref, CSS compliqué par dessus un HTML non valide. Je ne suis pas fan de ce patch.

#5 Updated by Frédéric Péters 8 months ago

Il y a du travail, ailleurs, pour faire évoluer le markup des formulaires d'authentic, #40159, mais avancer là-dessus ne doit pas empêcher de corriger les bugs maintenant, même si la manière est pas jolie.

#6 Updated by Thomas Noël 4 months ago

  • Status changed from Solution proposée to Solution validée

Validation du sparadrap.

#7 Updated by Frédéric Péters 4 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit fa69cfd879ce7d40df0d1baeedda5111cb806add
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon Apr 6 10:23:05 2020 +0200

    scss: style civility field on registration page (#41387)

#8 Updated by Frédéric Péters 4 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF