Bug #40712
villeurbanne, hauteur de champ fichier
0%
Description
Avec #40251 on laisse la hauteur du champ fichier augmenter pour l'accessibilité. Mais il y un height: 100% pour un des éléments et dans des circonstances particulières (#40711) ça va calculer ce 100% par rapport à un élément extérieur bien trop haut.
Fichiers
Demandes liées
Révisions associées
scss: force height of file input when it displays a thumbnail (#40712)
Historique
Mis à jour par Frédéric Péters il y a environ 4 ans
Le height: 100% vient à l'origine de #29317, mais ce ticket manque de captures d'écran pour montrer le problème.
Mais en gros, "On peut ne pas forcer de hauteur sur cette page, ou dimensionner l'image pour lui forcer sa hauteur."
L'option prise a été la seconde "dimensionner l'image pour lui forcer sa hauteur" sauf qu'avec le changement d'accessibilité, ça s'est transformé en "ne pas forcer de hauteur".
Mis à jour par Frédéric Péters il y a environ 4 ans
- Fichier 0001-scss-force-height-of-file-input-when-it-displays-a-t.patch 0001-scss-force-height-of-file-input-when-it-displays-a-t.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Ça devient compliqué tout ça, au point d'ajouter un commentaire pour m'y retrouver...
Mis à jour par Frédéric Péters il y a environ 4 ans
- Lié à Development #40251: upload de fichier, permettre à sa hauteur d'augmenter ajouté
Mis à jour par Frédéric Péters il y a environ 4 ans
- Lié à Bug #29317: taille de la vignette d'un fichier sur la page validation ajouté
Mis à jour par Frédéric Péters il y a environ 4 ans
Pour parler de la situation présente, le fileinfo { height: 100% } posait problème à Villeurbanne parce qu'il se trouvait correspondre à un élément parent dont la hauteur était (somehow) fixée (ce que les autres intégrations ne font pas, sans doute qu'à Villeurbanne c'est pour cette navigation qui "glisse"). Et donc, quand fileinfo était affiché (quand un fichier était sélectionné), la zone prenait la hauteur de ce parent. Dans les autres intégrations, comme il n'y avait pas de tel élément parent, le height: 100% était égal à height: auto, pas de soucis.
Le patch retire donc le height: 100%, corrige l'affaire à Villeurbanne.
Mais le height: 100% était là pour une raison, pour que les images reprises sur la page de validation soient limitées en hauteur. Cette partie ne marchait plus depuis le changement accessibilité qui a retiré le height fixe pour le remplacer par un max-height. Donc là, on refait les deux, 1/ on met un height fixe ce qui n'est ici pas un problème d'accessibilité parce qu'on n'a pas de texte à afficher, 2/ on met dans cette situation le height: 100% pour que la vignette soit redimensionnée (plutôt que coupée).
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
- Assigné à mis à Frédéric Péters
Go.
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit c876a38fdd399b602f660615ef53419cc262f112 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Fri Mar 13 10:00:21 2020 +0100 scss: force height of file input when it displays a thumbnail (#40712)
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
scss: force height of file input when it displays a thumbnail (#40712)