Projet

Général

Profil

Development #45365

suppression des <br class="clear">

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Thomas Jund (congés, retour le 29/04)
Version cible:
-
Début:
22 juillet 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Pour compenser le layout en float, quelques <br class="clear"> et <div style="clear:both"> ont été ajoutés au code.

  1. On est pas HTML valide
  2. leur présence pose des problèmes pour une migration vers flexbox ou grid (en devenant flex-item ou gris-item)

Je propose de les supprimer et de les remplacer par une class "clearfix" lorsqu'un fix est encore nécessaire.

Fichiers

0001-replace-dedicated-clear-float-tags-by-.clearfix-clas.patch (2,82 ko) 0001-replace-dedicated-clear-float-tags-by-.clearfix-clas.patch Thomas Jund (congés, retour le 29/04), 22 juillet 2020 16:39
0002-montpellier-replace-dedicated-clear-float-tags-by-.c.patch (3,04 ko) 0002-montpellier-replace-dedicated-clear-float-tags-by-.c.patch Thomas Jund (congés, retour le 29/04), 22 juillet 2020 16:39
0003-toodego-replace-dedicated-clear-float-tags-by-.clear.patch (925 octets) 0003-toodego-replace-dedicated-clear-float-tags-by-.clear.patch Thomas Jund (congés, retour le 29/04), 22 juillet 2020 16:39
0004-blois-2018-replace-dedicated-clear-float-tags-by-.cl.patch (713 octets) 0004-blois-2018-replace-dedicated-clear-float-tags-by-.cl.patch Thomas Jund (congés, retour le 29/04), 22 juillet 2020 16:39
0005-groupe-up-replace-dedicated-clear-float-tags-by-.cle.patch (703 octets) 0005-groupe-up-replace-dedicated-clear-float-tags-by-.cle.patch Thomas Jund (congés, retour le 29/04), 22 juillet 2020 16:39
0006-metz-metropole-2019-replace-dedicated-clear-float-ta.patch (749 octets) 0006-metz-metropole-2019-replace-dedicated-clear-float-ta.patch Thomas Jund (congés, retour le 29/04), 22 juillet 2020 16:39
0007-publik-replace-dedicated-clear-float-tags-by-.clearf.patch (622 octets) 0007-publik-replace-dedicated-clear-float-tags-by-.clearf.patch Thomas Jund (congés, retour le 29/04), 22 juillet 2020 16:39
0008-quimper-replace-dedicated-clear-float-tags-by-.clear.patch (875 octets) 0008-quimper-replace-dedicated-clear-float-tags-by-.clear.patch Thomas Jund (congés, retour le 29/04), 22 juillet 2020 16:39
0009-seine-et-marne-apa-replace-dedicated-clear-float-tag.patch (892 octets) 0009-seine-et-marne-apa-replace-dedicated-clear-float-tag.patch Thomas Jund (congés, retour le 29/04), 22 juillet 2020 16:39
0010-signal-publik-replace-dedicated-clear-float-tags-by-.patch (905 octets) 0010-signal-publik-replace-dedicated-clear-float-tags-by-.patch Thomas Jund (congés, retour le 29/04), 22 juillet 2020 16:39
0001-replace-dedicated-clear-float-tags-by-.clearfix-clas.patch (5,84 ko) 0001-replace-dedicated-clear-float-tags-by-.clearfix-clas.patch Thomas Jund (congés, retour le 29/04), 22 juillet 2020 16:40
0001-replace-dedicated-clear-float-tags-by-.clearfix-clas.patch (727 octets) 0001-replace-dedicated-clear-float-tags-by-.clearfix-clas.patch Thomas Jund (congés, retour le 29/04), 22 juillet 2020 16:40

Révisions associées

Révision b460a1fe (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a presque 4 ans

replace dedicated clear float tags by ".clearfix" class (#45365)

Révision f94856b7 (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a environ 2 ans

rouen: replace dedicated clear float tags by ".clearfix" class (#45365)

Historique

#1

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

Patchs pour publik-base-theme

#4

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

Pas tout regardé mais liege, pas rapport aux autres, pourquoi style="..." plutôt que class="clearfix" ?

+<div id="footer-text">
+<div id="footer-text" style="clear:both">
#5

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

Pas tout regardé mais liege, pas rapport aux autres, pourquoi style="..." plutôt que class="clearfix" ?

Par facilité.
la class .clearfix se place sur le parent qui possède des float.
Dans ce cas il n'y a pas de parent qui englobe #left et #right qui se trouve au dessus de #footer-text. Ajouter un wrapper .clearfix autour de #left et #right aurait pu casser les CSS.
Il aurait été plus propre de placer le clear both dans les CSS, mais comme ce theme n'est plus utilisé (remplacé par liege-pwa-2019 je crois) mais fait un patch rapide.

#6

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

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

Ok go.

#7

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

(et je préfère que tout soit réuni dans un unique commit, au cas où il faudrait reverter pendant tes congés).

#8

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

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

Commit Publik-base-theme

commit b460a1fe7450ee7c19bbd3319443344d9e012a0e (HEAD -> master, origin/master, origin/HEAD)
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Wed Jul 22 12:13:49 2020 +0200

    replace dedicated clear float tags by ".clearfix" class (#45365)
#9

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

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

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

Commits imio-publik-themes

commit a34654a204c7cc2397b34df116859b2927315a46 (HEAD -> master, origin/master, origin/HEAD, dev)
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Tue Aug 11 17:04:36 2020 +0200

    update publik-base-theme submodule

commit a85c6ace9b052606d8287041501273bb06fd6569
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Wed Jul 22 16:12:52 2020 +0200

    replace dedicated clear float tags by ".clearfix" class (#45365)
#11

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

Commit rouen-publik-theme

commit 298ba8c23de6de2d497794085bee3fa5d6e3b145 (HEAD -> master, origin/master, origin/HEAD)
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Wed Jul 22 16:17:45 2020 +0200

    replace dedicated clear float tags by ".clearfix" class (#45365)
#12

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

Commit rouen-publik-theme

Oubli de mettre à jour le submodule ici ?

#13

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

(et pareil nécessité de mettre à jour le submodule dans atreal-publik-themes ?)

#14

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

Oubli de mettre à jour le submodule ici ?

(et pareil nécessité de mettre à jour le submodule dans atreal-publik-themes ?)

Ce patch n'apporte aucune modif sur les thèmes Atreal, et une modif sans conséquence sur Rouen.
(Je peux mettre à jour le submodule mais c'est plus pour une mise à niveau qu'un besoin pour ce patch).

#15

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

Pour expliquer ce qui va se passer, la page login va être affichée avec

<div id="login-page" class="clearfix methods{{ blocks|length }}">

mais la classe clearfix n'existera pas si le sous-module n'a pas été mis à jour.

Boum.

#19

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

update publik-base-theme pour rouen

commit 6a6a11185eb099dac6727e1dc3c8aaa4d63d6cf4 (HEAD -> master, origin/master, origin/HEAD)
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Thu Aug 13 18:30:38 2020 +0200

    publik-base-theme submodule update

update publik-base-theme pour Atreal

commit 4faf5ce51ac1e45a31f4de57a81c03dee78a86b7 (HEAD -> master, origin/master, origin/HEAD)
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Thu Aug 13 18:32:50 2020 +0200

    publik-base-theme submodule update

Formats disponibles : Atom PDF