Making WordPress.org

Opened 19 months ago

Closed 19 months ago

Last modified 19 months ago

#6128 closed defect (bug) (fixed)

Meta: Use global header height variable to calculate Openverse iframe height

Reported by: sarayourfriend's profile sarayourfriend Owned by: sarayourfriend's profile sarayourfriend
Milestone: Priority: normal
Component: Openverse Keywords: has-patch
Cc:

Description

As described here https://github.com/WordPress/openverse-frontend/issues/834#issuecomment-1048022272 we need to use the new header height variable to fix the openverse iframe height.

Change History (5)

zackkrida commented on PR #62:


19 months ago
#2

@sarayourfriend as part of this could we also remove vertical overflow from the WP theme, to prevent multiple scrollbars and the wp.org footer from taking up search result screen real estate? body {overflow:hidden;} should do the trick.

sarayourfriend commented on PR #62:


19 months ago
#3

@WordPress/openverse Please review this when y'all have a chance this week. Thanks!

#4 @dd32
19 months ago

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

In 11656:

Openverse: Use global header height variable to calculate Openverse iframe height.

Props sarayourfriend, zackkrida, dhruvkb.
Closes https://github.com/WordPress/wordpress.org/pull/62.
Fixes #6128.

#5 @dd32
19 months ago

  • Component changed from WordPress.org Site to Openverse
Note: See TracTickets for help on using tickets.