Project

General

Profile

Actions
Copy link

Bug #16024

closed

Tools>User-admin is hard to "read"

Added by Martin Kutschker over 18 years ago. Updated about 14 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Jeff Segars
Category:
Backend User Interface
Target version:
-
Start date:
2006-04-10
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

There are no boxes around the grouped users any more. Now its hard to tell which setting belongs to which group of users.

There's no need to make borders around the complete table. It's IMHO sufficient to add horizontal borders on the TRs to get lines between the users.
(issue imported from #M3215)

Files

Download all files
2080217_user_admin.diff (1.25 KB) 2080217_user_admin.diff Administrator Admin, 2008-02-17 00:48
20080217_user_tools_cleanup.patch (12.6 KB) 20080217_user_tools_cleanup.patch Administrator Admin, 2008-02-18 00:32
20080218_user_tools_cleanup.patch (15.6 KB) 20080218_user_tools_cleanup.patch Administrator Admin, 2008-02-18 12:42
Actions
Copy link
 #1

Updated by Steffen Kamper almost 17 years ago

i made alternate row-classes so this will be affect old style and t3skin.
For t3skin i added one more color (bgcolor6)

patch is attached

Actions
Copy link
 #2

Updated by Benni Mack almost 17 years ago

I wanted to read through the file and found some ugly things in it and did a small cleanup (mostly added phpDoc headers where missing and replaced each-while with foreach loops). Attached is the patch. I haven't tested it yet but will do in the next days. if this patch is good, I'll submit it as a followup to your's in the core list, steffen.

Actions
Copy link
 #3

Updated by Jeff Segars almost 17 years ago

Hey Benni,
I think you're missing the actual patch. Just got svn status instead :)

Actions
Copy link
 #4

Updated by Benni Mack almost 17 years ago

sorry :) I was (and still am) in a hurry, that's why I haven't sent it to the core list yet. I want to make sure it really works ;-)

Actions
Copy link
 #5

Updated by Benni Mack almost 17 years ago

Hey guys,

I tested both patches and they did not work, as I need class="bgColor6" instead of "class="bgcolor6". I fixed this with a new patch and am going to send this patch to the core list.

Actions
Copy link
 #6

Updated by Jeff Segars almost 17 years ago

Changed bgColor6 to use the same background color as alternating list rows and committed to 4.2 branch.

Actions
Copy link

Also available in: Atom PDF