#9366 closed enhancement (fixed)
Permissions screen is confusing when there are large groups — at Version 10
Reported by: | batrick | Owned by: | |
---|---|---|---|
Priority: | high | Milestone: | 1.0 |
Component: | admin/web | Version: | |
Severity: | normal | Keywords: | bitesized patch review |
Cc: | franoleg@…, osimons, ryano@… | Branch: | |
Release Notes: |
Separated permissions and group memberships in the permission admin panel. |
||
API Changes: | |||
Internal Changes: |
Description
We use trac with approximately 10 groups and 150 users. Functionally, everything works fine but the permissions screen is confusing because it shows one row per user (so there are approximately 160 rows shown, including the rights for each group).
It would be a fine usability enhancements if group memberships were, in fact, shown by group. In this case, there would be only 10 lines for permissions, and 10 for group memberships.
Change History (12)
comment:1 by , 14 years ago
Keywords: | bitesized added |
---|---|
Milestone: | → unscheduled |
comment:2 by , 12 years ago
I'm submitting a patch which implements this proposal (screenshot included). Changes are primarily in the Genshi template, which now lists perms
and groups
as separate tbody
s. Code only places the perms
and the groups
apart.
Form logic was not modified.
There are some changes in the CSS that I made in order to put subheadings within the table (I wanted the column widths to be the same, hence one table instead of two, a la #9825). I feel these changes can be done better. Any feedback so I can improve the patch?
by , 12 years ago
Attachment: | t9366-r1.diff added |
---|
Permissions and group membership are displayed as two tables, instead of one (against trunk).
comment:3 by , 12 years ago
Keywords: | patch review added |
---|
comment:4 by , 12 years ago
Milestone: | unscheduled → 0.13 |
---|---|
Owner: | set to |
Priority: | normal → high |
That looks nice! Thanks for the patch.
comment:5 by , 12 years ago
Cc: | added |
---|
comment:6 by , 12 years ago
Cc: | added |
---|
comment:7 by , 12 years ago
Cc: | added |
---|
comment:8 by , 12 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Improved patch applied in [11054], together with some cleanup. In particular:
- XHTML doesn't allow multiple
<thead>
sections in a<table>
. - Empty
<body>
are forbidden as well. - Put the
<input>
inside the<label>
, allowing to remove theid=
attribute, and with the advantage that clicking between the checkbox and the label will actually trigger the checkbox.
Thanks Oleg, your patch was an excellent starting point. I like the new layout very much.
comment:9 by , 12 years ago
Owner: | changed from | to
---|
comment:10 by , 12 years ago
Release Notes: | modified (diff) |
---|
This sounds like a nice idea.