Projet

Général

Profil

Bug #36038

position de la croix de suppression d'un fichier

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
11 septembre 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Elle est posée à côté du nom de fichier mais quand le nom de fichier est trop long et bien elle n'apparait pas.

Ça pourrait aller en haut à droite, ça serait l'occasion aussi d'en élargir la zone cliquable.


Fichiers

uploads.png (26,6 ko) uploads.png Frédéric Péters, 11 septembre 2019 14:47
upload2.png (11,7 ko) upload2.png Frédéric Péters, 11 septembre 2019 14:52
0001-scss-move-remove-file-icon-to-the-top-right-corner-3.patch (991 octets) 0001-scss-move-remove-file-icon-to-the-top-right-corner-3.patch Frédéric Péters, 11 septembre 2019 14:55
upload3.png (8,31 ko) upload3.png Frédéric Péters, 11 septembre 2019 17:24
0001-scss-move-remove-file-icon-to-the-top-right-corner-3.patch (1,04 ko) 0001-scss-move-remove-file-icon-to-the-top-right-corner-3.patch Frédéric Péters, 27 février 2020 21:26

Révisions associées

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

scss: move "remove file" icon to the top right corner (#36038)

Historique

#1

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

#2

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

Premier avis.

Proposition qui permet de supprimer le fichier même si le nom de fichier est trop long mais

Le vrai problème initial est que le nom du fichier ne s'affiche pas en entier. Ce qui pose un problème ergonomique non négligeable. Si l'objectif est que l'utilisateur ai un feedback lui permettant d'être sûr que le fichier sélectionné est le bon. Permettre l'affichage du nom du fichier en entier est donc la solution première du problème. Déplacer le bouton de suppression est intéressant mais ne règle à mon avis que la moitié du problème.

2. ++ concernant la taille du bouton ou de la zone cliquable. En tactile la taille d'un bouton devrait au moins faire 30x30px

#3

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

Ma position est que si le nom du fichier est long c'est que ce n'est pas un nom qui a été choisi par l'usager, genre ici un YXLt..., ailleurs le timestamp de la photo prise par le mobile, et que du coup il ne dira de toute façon rien à l'usager.

#4

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

Ma position est que si le nom du fichier est long c'est que ce n'est pas un nom qui a été choisi par l'usager,

Donc tu pars du principe qu'un usager ne nommera jamais ses fichiers avec des noms du genre.

fete-de-la-pomme-de-terre-2019-souffelweyersheim-01.jpg

Et qu'il n'utilise jamais comme différentiation entre une liste de fichiers un système du numérotation en suffixe d'un nom très long ?

#5

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

Je propose de requalifier la justification comme étant que la position de la croix serait de toute façon plus naturelle en haut à droite, qui est une localisation usuelle pour fermer/faire disparaitre.

#6

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

(ça laissera la possibilité de discuter de l'affichage ou pas du nom de fichier ailleurs, ça permettra également d'amener dans un ticket public #33231).

#7

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

OK.

Et pourquoi pas, positionner le centre du bouton sur le coin top right plutôt que dans l'angle, pour visuellement différentier les informations du fichiers = dans le bloc, actions = un peu en dehors / superposé. Position qui serait encore plus usuelle non ?

#8

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

Sur l'angle même, mmm, comme crainte que ça commence à s'approcher un peu trop du libellé au-dessus, et oui un peu, mais ça me permet de voir aussi deux autres problèmes : ça oblige de retirer le overflow: hidden du bouton avec des conséquences que je ne mesure pas (au moins qu'il va falloir le mettre ailleurs pour ne pas avoir le nom du fichier sortir du cadre) et aussi comme l'icône FontAwesome n'a pas de fond, ça fait qu'à travers la croix on voit la bordure du champ.

#9

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

Donc proposition un peu compliqué à mettre en oeuvre avec le code existant et qui demanderait une refactorisation. Je n'ai pas pensé au label, car pour moi le module prends toute la largeur de la cellule parent. Mais en effet ce n'est pas le cas, il a un max-width.

Le coup de l'icone est pas mal aussi :). je pensais que c'était un icone croix blanche sur bloc couleur en border-radius.
Quelle idée de faire l'inverse ? FontAwesome propose des trucs vraiment inutiles.

J'ai besoin de monter en compétence sur le code existant avant de chercher à être force de proposition.

#11

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

Le problème vient d'être soulevé par imio (#37189), ça m'irait d'avancer ainsi.

#14

Mis à jour par Emmanuel Cazenave il y a environ 4 ans

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

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 a11565d0b92d285b6e749a8187c80977d5e461b4
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Sep 11 14:55:17 2019 +0200

    scss: move "remove file" icon to the top right corner (#36038)
#16

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