Modify

Opened 21 months ago

Last modified 21 months ago

#10740 new defect

[Patch] Checkbox columns are too wide on the Accounts: Cleanup page

Reported by: rjollos Owned by: hasienda
Priority: normal Component: AccountManagerPlugin
Severity: minor Keywords: web-UI CSS
Cc: Trac Release: 1.2

Description

In Trac 1.1.1dev (and probably some earlier versions), the column of checkboxes is too wide.


Attachments (3)

AccountsCleanup.png (27.3 KB) - added by rjollos 21 months ago.
AccountsCleanup2.png (22.1 KB) - added by rjollos 21 months ago.
th10740-r12491-1.patch (550 bytes) - added by rjollos 21 months ago.

Download all attachments as: .zip

Change History (10)

Changed 21 months ago by rjollos

Changed 21 months ago by rjollos

comment:1 Changed 21 months ago by rjollos

  • Summary changed from Checkbox columns are too wide on the Accounts: Cleanup page to [Patch] Checkbox columns are too wide on the Accounts: Cleanup page

The attached patch uses a class from admin.css, the same class that several other tables in the admin realm use, including the Ticket Components table. After this page, the checkboxes columns are the same width as I see on a Trac 0.12.3 installation.


Changed 21 months ago by rjollos

comment:2 Changed 21 months ago by hasienda

  • Keywords web-UI CSS added
  • Severity changed from normal to minor
  • Trac Release set to 1.2

I can't reproduce this, at least not with Trac-1.0, but I trust you, and the change looks non-intrusive. So I'll apply this right-way, and profit once I'll go forward towards Trac trunk again.

comment:3 Changed 21 months ago by rjollos

I'm curious as to what might have changed between Trac 1.0 and the current 1.1.dev, so I'll be digging deeper on this one and will report back here.

I noticed that the table on the Manage User Accounts page also uses the sel class for the header column of checkboxes, same as the Ticket Components table and others. If I remove the class from that table, I also get the larger column width in Trac 1.1.1dev.

comment:4 Changed 21 months ago by hasienda

This sounds like an explanation. Must have copied this from Trac for the older tables already due to similar experiences in the past, yes.

comment:5 Changed 21 months ago by hasienda

(In [12493]) AccountManagerPlugin: Keep table columns narrow in db clean-up admin page too, refs #10740.

This isn't visible in Trac 1.0 and below, but has been reported for latest
Trac trunk. Thanks to Ryan J Ollos for taking care.

comment:6 follow-up: Changed 21 months ago by rjollos

While working on t:#11003, I noticed that the sel class is applied to the td elements in the column for the tables in the admin realm, such as Ticket Components table. There doesn't appear to be any special CSS that applies to the td.sel selector, but I wonder if it would be good practice to apply the sel class to the td elements here as well. This would keep the tables consistent in the case of future CSS changes in Trac.

comment:7 in reply to: ↑ 6 Changed 21 months ago by rjollos

Replying to rjollos:

While working on t:#11003, I noticed that the sel class is applied to the td elements in the column for the tables in the admin realm, such as Ticket Components table. There doesn't appear to be any special CSS that applies to the td.sel selector, but I wonder if it would be good practice to apply the sel class to the td elements here as well. This would keep the tables consistent in the case of future CSS changes in Trac.

This is added (out of necessity) in the patch for #10745.

Add Comment

Modify Ticket

Action
as new .
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.