Make WordPress Core

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#25961 closed enhancement (fixed)

Appearance Themes: theme detailed view can use more of the viewport's height in some cases

Reported by: matveb's profile matveb Owned by: dd32's profile dd32
Milestone: 3.8 Priority: normal
Severity: normal Version: 3.8
Component: Customize Keywords: has-patch
Focuses: ui Cc:

Description

The overlay used to show theme details should have more flexible top and bottom margins. Specially when the window height is small, a lot of space is wasted.

Attachments (1)

25961.diff (728 bytes) - added by shaunandrews 10 years ago.
Replacing the fixed px spacing for the modal with percentages.

Download all attachments as: .zip

Change History (5)

@shaunandrews
10 years ago

Replacing the fixed px spacing for the modal with percentages.

#1 @SergeyBiryukov
10 years ago

  • Keywords ui-focus has-patch added
  • Milestone changed from Awaiting Review to 3.8

#2 @shaunandrews
10 years ago

I've rolled this patch into #26192 as that ticket affects the positioning of the modal as well.

#3 @dd32
10 years ago

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

In 26519:

Themes: Make the Theme Detail view div use more of the available viewport, Fix the Header and Footer in the viewport and only scroll the theme details, Prefix some more theme CSS classes. Props shaunandrews for the initial viewport patch. Fixes #25961, #26192

#4 @dd32
10 years ago

In 26520:

Themes: Indent the HTML for proper indentation after r26519 added an extra div. See #25961, #26192

Note: See TracTickets for help on using tickets.