Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#26354 closed defect (bug) (fixed)

Dashboard responsive columns don't work correctly when you set a specific number of columns.

Reported by: westi's profile westi Owned by: nacin's profile nacin
Milestone: 3.8 Priority: lowest
Severity: trivial Version: 3.8
Component: Administration Keywords: needs-patch has-patch
Focuses: Cc:

Description

We have CSS that supports specific numbers of columns of dashboard widgets as well as responsive ones.

However if you use wp_dashboard and related APIs to create your own page of widgets in a plugin with a fixed number or selectable number of columns we never output the css classes that disable the responsive CSS.

Also the CSS for the unresponsive fixed columns is very crowded and loses the separation between the columns.

Attachments (3)

26354-php.diff (686 bytes) - added by westi 11 years ago.
Simple patch to output the class the CSS supports
26254-php.2.diff (686 bytes) - added by westi 11 years ago.
A patch with proper CSS usage, thanks to kovshenin for pointing out my stupidity
26354.diff (1.6 KB) - added by kovshenin 11 years ago.

Download all attachments as: .zip

Change History (9)

@westi
11 years ago

Simple patch to output the class the CSS supports

#1 @westi
11 years ago

That patch makes 2 column mode work on a wide screen for me when I have a plugin that uses wp_dashboard() and has a 1/2 column toggle in screen options.

CSS needs some work though so there is still separation between the columns.

@westi
11 years ago

A patch with proper CSS usage, thanks to kovshenin for pointing out my stupidity

#2 @nacin
11 years ago

  • Priority changed from normal to lowest
  • Severity changed from normal to trivial

I don't know if we should be supporting wp_dashboard() being used in a plugin.

#3 @nacin
11 years ago

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

In 26617:

Include columns-# CSS class in wp_dashboard() for when the screen supports it.

props westi.
fixes #26354.

@kovshenin
11 years ago

#4 @kovshenin
11 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

In 26354.diff: use more specific CSS selectors to target metabox containers with the columns screen setting, otherwise they're pretty useless with our responsive dash media queries.

#5 @nacin
11 years ago

Seems like all of this CSS (and [26617]) should just be removed outright, no?

#6 @nacin
11 years ago

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

In 26756:

Proper CSS specificity for columns-based dashboard grids.

props kovshenin.
fixes #26354.

Note: See TracTickets for help on using tickets.