Make WordPress Core

Opened 4 months ago

Last modified 7 days ago

#60462 new enhancement

Twenty Twenty: Cover template overflows browser bar on mobile

Reported by: twiddler's profile twiddler Owned by:
Milestone: Future Release Priority: normal
Severity: trivial Version:
Component: Bundled Theme Keywords: has-patch needs-testing
Focuses: css Cc:

Description

Hey there!

Twenty Twenty's cover template fills the whole background with the article image. This works as expected on desktop. On mobile however, some content gets covered by the browser bar.

Replacing vh with dvh fixes that:

.screen-height {
   min-height: 100dvh;
}

Attachments (1)

60462.diff (1.2 KB) - added by twiddler 4 months ago.

Download all attachments as: .zip

Change History (5)

@twiddler
4 months ago

This ticket was mentioned in PR #6062 on WordPress/wordpress-develop by twiddler.


4 months ago
#1

  • Keywords has-patch added

Hey there! :wave:

Twenty Twenty's cover template fills the whole background with the article image. This works as expected on desktop. On mobile however, some content gets covered by the browser bar.

Replacing vh with dvh fixes that:

.screen-height {
   min-height: 100dvh;
}

Trac ticket: https://core.trac.wordpress.org/ticket/60462

#2 @sabernhardt
4 months ago

  • Component changed from Themes to Bundled Theme
  • Focuses css added; ui removed
  • Keywords changes-requested removed
  • Version trunk deleted

#3 @karmatosed
3 months ago

  • Milestone changed from Awaiting Review to Future Release

#4 @karmatosed
7 days ago

  • Keywords needs-testing added

It would be great to get some testing on this to ensure we discover if this is still an issue and if we want to progress.

Note: See TracTickets for help on using tickets.