WordPress.org

Make WordPress Core

Opened 2 years ago

Closed 2 years ago

#23581 closed defect (bug) (wontfix)

Twenty Thirteen: Header takes up too much space on mobile screens

Reported by: kwight Owned by:
Milestone: Priority: low
Severity: normal Version: 3.6
Component: Bundled Theme Keywords: has-patch
Focuses: Cc:

Description

The header has a minimum height of 230px pixels at all widths, including mobile. A smaller header could be combined with the reduced site title font to maximize space on small screens.

The header background image is forced to background-position: center in the patch, to increase chances of seeing something interesting at such a short image height.

Attachments (1)

23581.diff (509 bytes) - added by kwight 2 years ago.

Download all attachments as: .zip

Change History (5)

@kwight2 years ago

comment:1 @lancewillett2 years ago

  • Keywords ui-feedback added
  • Milestone changed from Awaiting Review to 3.6
  • Priority changed from normal to low

I'd prefer to leave the design as-is -- but will loop back with Joen for a double-check on the design decision.

comment:2 @kwight2 years ago

Just throwing it out there; it's always nice to see more content than header when arriving at a site on mobile.

comment:3 @karmatosed2 years ago

I would second really this is a design decision. On an iPhone you get the header, menu/search and most of first post which to me seems reasonable and a design decision rather than issue.

comment:4 @obenland2 years ago

  • Keywords ui-feedback removed
  • Milestone 3.6 deleted
  • Resolution set to wontfix
  • Status changed from new to closed

Joen and I talked about it in the initial development phase and we agreed it to leave it the way it is. The header is only 230px high, which makes it the second-smalles among Twenties, and more importantly we didn't want to crop the background image horizontally, which would make it even harder than it is now for users to find a fitting image.

Note: See TracTickets for help on using tickets.