[#6674] Visual enhancement at perms.inc template to show long group names
Summary Visual enhancement at perms.inc template to show long group names
Queue Kronolith
Queue Version HEAD
Type Enhancement
State Rejected
Priority 1. Low
Owners
Requester almarin (at) um (dot) es
Created 05/02/2008 (9 days ago)
Due
Updated 05/02/2008 (9 days ago)
Assigned 05/02/2008 (9 days ago)
Resolved 05/02/2008 (9 days ago)
Attachments files.tar.gz Download
addnig_groups.png Download
Milestone
Patch 1

History
05/02/2008 Chuck Hagenbuch State ⇒ Rejected
 
05/02/2008 almarin (at) um (dot) es Comment #8 Reply to this comment
Ok, I will wait for the live-box search then :)
05/02/2008 Chuck Hagenbuch Comment #7
Patch ⇒ 1		 Reply to this comment
I feel like shorter group names are a more likely case than long ones, so I'm inclined to leave it as-is.
05/02/2008 almarin (at) um (dot) es Comment #6 Reply to this comment
I think limiting with css would be a functional limitation, because given a deep subgroup, you wouldn't be able to know which one you are selecting.

I have tried with short groups and you are right, looks weird. But with a too long combo and checkboxes displaced to right looks bad aswell.

After all my proposition was aesthetic too, so the question would be: which one is te ugliest for you? It is up to you, you are the boss
05/02/2008 Chuck Hagenbuch Comment #5
State ⇒ Feedback
Taken from Chuck Hagenbuch		 Reply to this comment
Ugh. This looks pretty bad if the group names are short. How about just limiting the size of the select box with a css width?
05/02/2008 Chuck Hagenbuch Comment #4
State ⇒ Assigned
Assigned to Chuck Hagenbuch		 Reply to this comment
Okay. We should replace this with a live-search box at some point in the future, but for now, this seems fine. Thanks.
05/02/2008 almarin (at) um (dot) es Comment #3
New Attachment: addnig_groups.png Download		 Reply to this comment
Well, I can give you an example. See this new image attached, please. I added the longest group we have. With the table structure proposed, some elements are pushed down a bit, but the main idea is remaining checkboxes visible.
05/02/2008 Chuck Hagenbuch Comment #2
State ⇒ Feedback		 Reply to this comment
What about when you add those groups? Does that push the page over also?
05/02/2008 almarin (at) um (dot) es Comment #1
New Attachment: files.tar.gz Download
Patch ⇒
Milestone ⇒
Queue ⇒ Kronolith
Summary ⇒ Visual enhancement at perms.inc template to show long group names
Type ⇒ Enhancement
Priority ⇒ 1. Low
State ⇒ New		 Reply to this comment
We have some hierarchical groups defined in our Horde installation. Since Horde uses to print that hierarchy as group1:subgroup11:subsubgroup111:..., it is easy to get group names very very long.

That causes permission templates show checkboxs beyond the popup window limits, because of the group combo-box.

I think that problem could be very typical mainly because of the printing schema described.

I attach some images to explain better what I mean. And I attach a patch for kronolith/templates/perms/perms.inc to propose a little modification that could be done to handle the problem (similar modification should be done in other templates like horde/templates/shares/edit.inc)

Thanks.