Projet

Général

Profil

Bug #40712

villeurbanne, hauteur de champ fichier

Ajouté par Frédéric Péters il y a environ 4 ans. Mis à jour il y a environ 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
13 mars 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Lié à Intégrations graphiques Publik - Development #40251: upload de fichier, permettre à sa hauteur d'augmenterFermé27 février 2020

Actions
Lié à Intégrations graphiques Publik - Bug #29317: taille de la vignette d'un fichier sur la page validationFermé21 décembre 2018

Actions

Révisions associées

Révision c876a38f (diff)
Ajouté par Frédéric Péters il y a environ 4 ans

scss: force height of file input when it displays a thumbnail (#40712)

Révision dffc069e (diff)
Ajouté par Frédéric Péters il y a environ 4 ans

scss: force height of file input when it displays a thumbnail (#40712)

Historique

#1

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".

#2

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

Ça devient compliqué tout ça, au point d'ajouter un commentaire pour m'y retrouver...

#3

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é
#5

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é
#6

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).

#7

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.

#8

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)
#9

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

Formats disponibles : Atom PDF