Tickets :: [#6674] Visual enhancement at perms.inc template to show long group names

6.0.0-beta1

9/24/25

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 (6354 days ago)
Due
Updated	05/02/2008 (6354 days ago)
Assigned	05/02/2008 (6354 days ago)
Resolved	05/02/2008 (6354 days ago)
Milestone
Patch	Yes

05/02/2008 06:14:17 PM	Chuck Hagenbuch	State ⇒ Rejected

05/02/2008 06:01:31 PM	almarin (at) um (dot) es	Comment #8	Reply to this comment
Ok, I will wait for the live-box search then :)

05/02/2008 05:58:13 PM	Chuck Hagenbuch	Comment #7 Patch ⇒ Yes	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 05:56:43 PM	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 05:43:38 PM	Chuck Hagenbuch	Comment #5 Taken from Chuck Hagenbuch State ⇒ Feedback	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 05:29:14 PM	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 03:58:17 PM	almarin (at) um (dot) es	Comment #3 New Attachment: addnig_groups.png	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 03:46:03 PM	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 09:30:50 AM	almarin (at) um (dot) es	Comment #1 Priority ⇒ 1. Low State ⇒ New New Attachment: files.tar.gz Patch ⇒ No Milestone ⇒ Queue ⇒ Kronolith Summary ⇒ Visual enhancement at perms.inc template to show long group names Type ⇒ Enhancement	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.