Feature #4365

Feature #918: Review right management

Display profile choice as matrix

Added by webmyster 11 months ago. Updated 2 months ago.

Status:Closed Start date:06/04/2013
Priority:Normal Due date:
Assignee:webmyster % Done:

100%

Category:Ergonomics
Target version:0.85

Description

L'affichage du choix des droits associés à un profile ne satisfait pas grand monde. Je propose de passer par une matrice avec des "checkbox", ce qui me semble plus naturel.

Le patch joint propose une première approche (cf. onglet Administration). Cependant, il y a encore des trucs à améliorer : il serait bon d'incliner les droits en haut. Il faudrait voir si on fait un tableau pour chaque groupe de droit ou si on fait un unique tableau par page. La seconde solution semble plus jolie car les colonne sont bien alignées. En revanche, cela créée des matrices "creuses". Mais si on incline les en-têtes et on réduit la largeur des colonnes à celle des "checkbox" ou à la hauteur du texte, la matrice semblera beaucoup moins vide.

Nous devrions pouvoir réutiliser le même type de matrice pour les cycles de vie des ticket, des problèmes et des changements.

svn.diff (10.9 kB) webmyster, 06/04/2013 07:12 pm

svn.diff - Essai en découpant 1 label par colonne. (1.7 kB) webmyster, 07/19/2013 12:12 pm

Associated revisions

Revision 21317
Added by webmyster 9 months ago

[0.85] Work for #4365. New framework for Asset, Administration and Life Cycles tabs.

Revision 21372
Added by webmyster 9 months ago

[0.85] Work for #4365: Management/Tools

Revision 21374
Added by webmyster 9 months ago

[0.85] Work for #4365: Assistance

Revision 21376
Added by webmyster 9 months ago

[0.85] Work for #4365: Setup

Revision 21405
Added by webmyster 9 months ago

[0.85] Work for #4365: simplified interface

Revision 22443
Added by moyo 3 months ago

add check all for admin life cycles see #4365

History

Updated by webmyster 9 months ago

Il faut encore déterminer si nous basculons tous les Dropdown::showYesNo() sur l'utilisation d'un checkbox plutôt que Dropdown::showFromArray().

En phase transitoire, nous pourrions ajouter une option 'use_checkbox' passée dans le tableau des paremêtres (dernier argument de la méthode). Comme cela, chacun pourra choisir entre une dropdown ou une case à cocher.

Nous pourrions maintenir les deux affichages différents : un des critères de choix pourrait être le nombre de cases à cocher dans le formulaire. En effet, au delà d'une demi-douzaine de valeurs booléenne à choisir, la dropdown risque d'être fastidieuse et nous pourrions basculer sur des checkbox.

Updated by moyo 9 months ago

Perso je serais pour tout basculer vers des checkbox.
On peut garder une option pour utiliser la dropdown si besoin.

Je pense qu'il faut un affichage le plus uniforme possible. Docn quelquesoit le nombre de valeurs à choisir i lfaut toujours que ce soit le même type d'affichage (checkbox pour moi)

Updated by moyo 9 months ago

Probleme sur la partie admin.
Check for upgrade pour 3 elements alors qu'il y en a qu'un. Je pense même valeur de droit mais pas même droit.
Il doit falloir splitter le tableau en plusieurs.
Idem pour les règles, la colonne Business rules for ticket (entity parent) fait bizarre.

Updated by webmyster 9 months ago

Effectivement, ils ont le même identifiant.
Nous avons deux possibilités pour résoudre ce problème : on sépare les tableaux, ou on fait des colonnes distincts.
On peut se baser sur le label associé pour savoir s'il faut des colonnes distincts. Par exemple, l'identifiant 1024 a trois "labels" différents :
  • __('Update method for user authentication and synchronization') pour User ;
  • __('Update helpdesk parameters') pour Entity ;
  • __('Check for upgrade') pour Backup.

Peut-on partir de ce principe pour dissocier les droits qui ont le même identifiant, mais pas la même signification ?

Updated by moyo 9 months ago

Il faut voir ce que cela donne au niveau affichage.
J'ai peut que séparer les colonnes engendre un nombre énorme de colonnes.

Updated by webmyster 9 months ago

L'un n'empêche pas l'autre. Si on trouve que le tableau est trop large, on peut le découper.
j'ai attaché un patch qui sépare en fonction des noms des labels.
Dans le cas des droits d'aministration, le tableau est trop large, il faudrait le redécouper.

Updated by webmyster 9 months ago

Je n'ai vraiment pas de mémoire !
C'est justement le propos de ce que nous disions : pour éviter que le tableau ne soit trop large, il faudrait incliner les labels des colonnes de 45°. Mais mes compétences en CSS sont trop limitées pour que je puisse le faire.

Updated by webmyster 9 months ago

  • % Done changed from 0 to 90

Updated by moyo 3 months ago

Que reste t'il à faire ?

Updated by webmyster 3 months ago

Pour moi, il ne reste rien à faire. Mais je peux avoir loupé des trucs quelque part ...
Peut-être faudrait-il ajouter la possibilité sélectionner/déselectionner tout dans les cycles de vie ?

La lisibilité est meilleure avec le highlight colonne/ligne introduit par le stylesheet. Qu'en pensent ceux qui disaient avoir des problèmes pour lire le tableau ?

Updated by moyo 3 months ago

webmyster wrote:

Peut-être faudrait-il ajouter la possibilité sélectionner/déselectionner tout dans les cycles de vie ?

Done

Updated by moyo 3 months ago

  • Status changed from Assigned to Resolved
  • % Done changed from 90 to 100

Updated by moyo 2 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF