WordPress.org

Make WordPress Core

Opened 3 weeks ago

Last modified 3 weeks ago

#48436 assigned defect (bug)

Media: fix the layout of Gallery settings in the media modal dialog

Reported by: afercia Owned by: afercia
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 3.9
Component: Media Keywords: has-screenshots needs-design-feedback
Focuses: ui Cc:
PR Number:

Description

In WordPress 3.8, the gallery settings in the media modal right sidebar were well aligned:

http://cldup.com/eEOQrDgtO6.png

The alignment broke pretty soon in WordPress 3.9, see [27487] / #26631. The CSS was changed to adjust the styling for the audio / video playlists settings but it broke the gallery details settings:

Screenshot from WordPress 3.9:

http://cldup.com/EP_dN25J_v.png

That's because the playlist settings are not intended to be aligned in two "columns":

http://cldup.com/9djN13Naos.png

The gallery settings are fully operable, so this is just a minor visual glitch.

Regardless, the layout is broken since March 2014 and still needs a fix on current trunk, even after the many changes to the media modal form controls markup and CSS, see [45499] / #47122.

http://cldup.com/XfzKug_020.png

Change History (2)

#1 @afercia
3 weeks ago

  • Keywords needs-design-feedback added

It appears this CSS ruleset:

.collection-settings .setting span {
    min-width: inherit;
}

is no longer necessary. Removing it would restore the "two columns" layout illustrated in the first screenshot above. The collection-settings selector is only used for galleries and playlists settings in the right sidebar, see the templates gallery-settings and playlist-settings. Removing that min-width: inherit won't affect anything else.

However, instead of restoring the two columns. I'd like to propose to stack the labels and form controls vertically (except for the checkbox). Two columns in a so limited space within the right sidebar proved to be problematic especially with longer translations.

#2 @afercia
3 weeks ago

  • Owner set to afercia
  • Status changed from new to assigned
Note: See TracTickets for help on using tickets.