Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#21985 closed defect (bug) (fixed)

Wrap .subsubsub for long list of views

Reported by: deltafactory's profile deltafactory Owned by: nacin's profile nacin
Milestone: 3.5 Priority: normal
Severity: trivial Version: 3.5
Component: Administration Keywords: has-patch commit
Focuses: Cc:

Description

The Views area of the WP List Table is styled not to wrap. This is a problem when the list of views is long. (e.g. an extended list of user roles on the Users page)

The following patch allows for wrapping while preventing wrapping within the <li> itself, so that the associated counter remains.

Most users should not notice a change to the appearance of this screen.

Attachments (8)

wp-admin.dev.css.patch (412 bytes) - added by deltafactory 12 years ago.
CSS patch for .subsubsub and .subsubsub li
views-before.png (13.1 KB) - added by deltafactory 12 years ago.
Views area before
views-after.png (13.2 KB) - added by deltafactory 12 years ago.
Views area after
wp-admin.css.patch (400 bytes) - added by deltafactory 12 years ago.
New patch file based on trunk, using renamed CSS file
chrome.png (21.7 KB) - added by ocean90 12 years ago.
21985.patch (329 bytes) - added by ocean90 12 years ago.
21985.2.patch (641 bytes) - added by SergeyBiryukov 12 years ago.
21985.3.patch (2.0 KB) - added by ocean90 12 years ago.
Refreshed

Download all attachments as: .zip

Change History (18)

@deltafactory
12 years ago

CSS patch for .subsubsub and .subsubsub li

#1 @nacin
12 years ago

A screenshot would help.

@deltafactory
12 years ago

Views area before

@deltafactory
12 years ago

Views area after

@deltafactory
12 years ago

New patch file based on trunk, using renamed CSS file

#2 @deltafactory
12 years ago

Screenshots added.

#3 @nacin
12 years ago

  • Keywords commit added
  • Milestone changed from Awaiting Review to 3.5

#4 @nacin
12 years ago

  • Owner set to nacin
  • Resolution set to fixed
  • Status changed from new to closed

In [21987]:

Avoid wrapping for individual .subsubsub view names, rather than the entire list of views, in case there are many. props deltafactory, fixes #21985.

@ocean90
12 years ago

@ocean90
12 years ago

#5 follow-up: @ocean90
12 years ago

  • Keywords commit removed
  • Resolution fixed deleted
  • Status changed from closed to reopened

Not quiet sure what's going on here, but [21987] breaks the list in Chrome @ OS X.

http://core.trac.wordpress.org/raw-attachment/ticket/21985/chrome.png

21985.patch fixes it for me.

Last edited 12 years ago by SergeyBiryukov (previous) (diff)

#6 @deltafactory
12 years ago

@ocean90: Just curious, what was the behavior in OS X/Chrome before? Original no-wrap behavior, or something else?

#7 in reply to: ↑ 5 @SergeyBiryukov
12 years ago

Replying to ocean90:

[21987] breaks the list in Chrome @ OS X.

Reproduced in Chrome on Windows as well.

Tested 21985.patch in Firefox 15, Chrome 22, IE 7, IE 8, Opera 12. IE 7 still needs display: inline (21985.2.patch), looks good otherwise.

#8 @Ipstenu
12 years ago

21985.2.patch works for Chrome OS X too.

@deltafactory - It was the original no-wrap.

@ocean90
12 years ago

Refreshed

#9 @ocean90
12 years ago

  • Keywords commit added
  • Type changed from enhancement to defect (bug)

#10 @nacin
12 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In 22404:

Use inline-block for wrapping filter links. props ocean90, SergeyBiryukov. fixes #21985.

Note: See TracTickets for help on using tickets.