#3921 closed enhancement (duplicate)
[PATCH] Permissions - update sort + compact permissions display
Reported by: | Owned by: | Christian Boos | |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | admin/web | Version: | 0.10 |
Severity: | minor | Keywords: | patch |
Cc: | Branch: | ||
Release Notes: | |||
API Changes: | |||
Internal Changes: |
Description
I use webadmin to manage a fair number of users, and have created this patch to address 2 issues.
- The permissions for a subject were not sorted
- The list was quite long
The patch :
- Sorts by subject+permission, giving an ordered list inside the permissions
- Combines all permission checks into a single row per subject, reducing size
Issues: The only issue is that the size of the individual permission div is 150, so if the perm name/group name becomes longer it could break the layout a little.
- Andrew
Attachments (6)
Change History (15)
by , 18 years ago
Attachment: | sort-and-perm-display-enhancement.patch added |
---|
by , 18 years ago
Attachment: | screenshot-perm-grouping.gif added |
---|
Screenshot of what the change looks like
by , 18 years ago
Attachment: | sort-and-perm-display-enhancement.2.patch added |
---|
(new) patch for perm.py&admin_perm.py, now with code style
comment:1 by , 18 years ago
Can you please make a patch against source:sandbox/webadmin-genshi, instead?
You use typical ClearSilver constructions, like:
<?cs if:(firstRun=="0")?></td></tr><?cs /if ?> ...
that don't map well to Genshi, so I'm personally not in favor of extending further the Clearsilver WebAdmin code base in a way that makes it painful to port to Genshi.
See TracDev/ApiChanges/0.11#MigratingawayfromClearsilver for further details.
Other than that, the change looks ok to me.
comment:2 by , 18 years ago
I will give the template port a shot once I can get a copy of the new one up and running. I was wondering if I should do that, but since I am actually using .10 in production (on my local dev box!) I just wrote it against that.
I'm going to go looking for how to convert to the new version w/o blowing things up.
comment:3 by , 18 years ago
Here is the new patch for Genshi. The default sort now appears fine, so I did not modify perm.py. The CSS and JS have both been modified slightly. You can now check a box by clicking on its text.
Are there any CSS revisions in the works? We can comfortably fit 3 columns of permissions if the add boxes on the right don't get bigger than they need to be (max-width? inside a wrapper div?)
by , 18 years ago
Attachment: | perm-display-grouping-genshi.patch added |
---|
For Genshi. Template, css, and js.
comment:4 by , 18 years ago
Milestone: | → 0.11 |
---|---|
Owner: | changed from | to
Thanks!
I've not verified it, but wouldn't <label for="$p.key">$p.action</label>
comment:5 by , 18 years ago
I always forget about labels, since they don't have much use for the other inputs. I am altering the patch to use labels instead. I usually put the 'hand' over the checkbox text but I have removed it for this rev since that means no other files are modified.
Could someone do me a favor and delete all attachments other than the latest. I am anon, so I can't even overwrite the previous patch.
by , 18 years ago
Attachment: | perm-display-grouping-genshi2.patch added |
---|
by , 18 years ago
Attachment: | perm-display-grouping-genshi3.patch added |
---|
Genshi final. v2 was the wrong file! Only diff is the .JS file
comment:6 by , 18 years ago
Keywords: | patch added |
---|
comment:7 by , 18 years ago
Will this be addressed, or do I need to reverify and submit as a patch under a different category?
comment:8 by , 18 years ago
Resolution: | → duplicate |
---|---|
Status: | new → closed |
Resumitted as #4873 now that it is in trunk.
comment:9 by , 18 years ago
Milestone: | 0.11 |
---|
patch for perm.py and templates/admin_perm.py