Project

General

Profile

Bug #60893

Affichage foireux des tableaux sur safari 13/14

Added by Emmanuel Cazenave 4 months ago. Updated 4 months ago.

Status:
Fermé
Priority:
Normal
Target version:
-
Start date:
20 Jan 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Cf Capture, reproduit en 13 et 14.


Files

Associated revisions

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

scss: do not use fixed table layout for responsive tables (#60893)

It's buggy on webkit when combined with grid placement of fields.

History

#2

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

  • Assignee set to Frédéric Péters

https://demarches-validation.test.entrouvert.org/tests/test-or-compatibilite-safari/

Le problème apparait aussi avec Epiphany ("Web") 41.3, pratique.

#3

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

C'est table-layout: fixed qui amène une série de problèmes avec webkit.

#4

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

Et notre utilisation vient à la base de #41734 pour un rendu "responsive" des tableaux, mais c'est une propriété qu'on pose effective uniquement pour l'affichage "desktop", justement pas dans le cas mobile, c'est curieux.

(il n'y a rien dans le ticket expliquant le pourquoi de cette partie du patch).

#5

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

2 patchs, 1/ ce côté publik-base-theme pour retirer le table-fixed: layout qui pose problème; 2/ côté w.c.s. pour ne plus forcer width: auto sur un tableau posé en grid (parce que la grid pose déjà width: 100% sur le tableau et on veut garder ça).

Formellement sur des réajustements successifs de largeur de tableau (mais c'est juste quelque chose qu'on fait pour tester en redimensionnant une fenêtre, en pratique ça doit peut arriver), il y a une différence après un passage "desktop" -> "mobile" -> "desktop", où un tableau dans un champ grid aura gagné un attribut explicite sur l'élément "width: 100%", qu'il avait avant uniquement via classe grid, et donc changement de poids de la propriété face aux sélecteurs et on pourrait imaginer des combinaisons ultérieures où ça aurait une incidence mais on pourrait aussi ne pas les imaginer parce que c'est bien peu probable.

#6

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

capture avec en fond webkit, à gauche en mobile firefox et à droite en mobile webkit à nouveau.

#8

Updated by Thomas Noël 4 months ago

On a aussi des grid-x-y sur les cellules combo, et certaines affichent des tableaux : ça ne va pas jouer là bas ?

#9

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

Non la classe "responsive table widget" ça arrive uniquement côté wcs.

#10

Updated by Thomas Noël 4 months ago

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

Allons-y alors !

... et j'en ai profité pour faire un « apt install epiphany-browser » ce qui permet, avec Firefox et Chromium, d'avoir sur une Debian les 3 grands moteurs de rendu HTML actuels : Gecko, Blink et WebKit/KHTML.

#11

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

  • Status changed from Solution validée to Résolu (à déployer)
commit 5f0ce98fa9e23ba0b9a9fe216fa1906d7af47f27
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Jan 21 09:01:39 2022 +0100

    scss: do not use fixed table layout for responsive tables (#60893)

    It's buggy on webkit when combined with grid placement of fields.
#12

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

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

Updated by Transition automatique 2 months ago

Automatic expiration

Also available in: Atom PDF