Projet

Général

Profil

Development #60271

Pour le select2 "ajouter au rôle" préciser qu'on peut chercher rôle ou utilisateur

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
04 janvier 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Comme en cliquant sur le champ elle se déplie en affichant des rôles on peut avoir l'impression qu'elle ne sert qu'aux rôles.

Je ne sais pas si ça peut se faire sous forme de placeholder à l'intérieur du select2 (je n'ai pas cherché le fond de django-select2) ni même si ce serait le meilleur endroit, ça peut peut-être juste être une ligne d'explication posée sous le champ. ? (maquette en exemple attachée)


Fichiers

Révisions associées

Révision f54ae16a (diff)
Ajouté par Valentin Deniaud il y a plus de 2 ans

manager: add placeholder and minimum input length to role member add field (#60271)

Révision 94f10efc (diff)
Ajouté par Valentin Deniaud il y a plus de 2 ans

manager: translate select2 messages in role member add field (#60271)

Historique

#1

Mis à jour par Valentin Deniaud il y a plus de 2 ans

  • Assigné à mis à Valentin Deniaud

Ouep, placeholder et exemple de rendu : https://select2.org/placeholders#single-select-placeholders

De plus on peut demander à taper plus de caractères au lieu d'afficher les premiers rôles : https://select2.org/searching#minimum-search-term-length

Et je découvre aussi sur l'exemple de champ avec placeholder qu'on peut avoir des groupes d'options, ce qui pourrait permettre d'avoir une indication visuelle sympa quand les options contiennent à la fois des rôles et des utilisateurs.

#3

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

Je relis/teste.

#4

Mis à jour par Valentin Deniaud il y a plus de 2 ans

Et dernier truc, messages select2 pas traduits.

#5

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

  • Statut changé de Solution proposée à En cours
  • Assigné à changé de Benjamin Dauvergne à Valentin Deniaud

Je crois que select2 n'aime pas du tout le combo pagination + arbre, au début les rôles sont bien sous le chapitre "Rôles" puis quand on scrolle (j'ai une base avec beaucoup de rôles et d'utilisateurs factices) tout se retrouve au premier niveau, arrivé en boute de liste des rôles, les utilisateurs arrivent et même comportement en fin de liste.

Soit on pense que personne ne va paginer et l'important c'est la présentation des premiers résultat et on supprime la pagination; soit on pense que la pagination est plus importante et on trouve un moyen d'ordonner les résultats des rôles et des utilisateurs, ça veut dire avoir un unique curseur qui en produit ensuite deux coté serveurs pour la pagination et trier les résultats mélangés utilisateurs+rôles et coté IHM un indicateur du type de résultat (une petite icône? un petit bonhomme pour un utilisateur, plusieurs pour un rôle).

#6

Mis à jour par Valentin Deniaud il y a plus de 2 ans

Benjamin Dauvergne a écrit :

au début les rôles sont bien sous le chapitre "Rôles" puis quand on scrolle tout se retrouve au premier niveau

Zut, j'avais même pas vu qu'il y avait une indentation pour signifier l'appartenance à un groupe. Effectivement select2 ne propose rien nativement pour faire ça proprement.

Du coup avançons avec les deux autres patches, je vais ouvrir un autre ticket spécifiquement pour l'amélioration de la présentation des résultats.

#7

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

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

Ok.

#8

Mis à jour par Valentin Deniaud il y a plus de 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 94f10efcdb0ca6699168acf6613c1433bc90456c
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Wed Jan 5 15:17:12 2022 +0100

    manager: translate select2 messages in role member add field (#60271)

commit f54ae16aad5489a94b0cd43f703d0489541257ac
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Wed Jan 5 14:45:39 2022 +0100

    manager: add placeholder and minimum input length to role member add field (#60271)
#9

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

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

Mis à jour par Transition automatique il y a environ 2 ans

Automatic expiration

Formats disponibles : Atom PDF