WordPress.org

Make WordPress Core

Opened 9 months ago

Closed 8 months ago

#24914 closed defect (bug) (fixed)

Twenty Fourteen: Space above the fixed bar in FF

Reported by: DrewAPicture Owned by: lancewillett
Milestone: 3.8 Priority: normal
Severity: normal Version: 3.8
Component: Bundled Theme Keywords: has-patch
Focuses: Cc:

Description

Seems like we're getting some extra space above the fixed bar when scrolling in Firefox. It doesn't do it when in the Customizer, and seems OK in Chrome, so this may be localized to this browser.

Screencast for your viewing displeasure: http://screencast.com/t/9ywPneKvD

Attachments (3)

chrome.jpg (162.4 KB) - added by MikeHansenMe 8 months ago.
screenshot of space using chrome
24914.diff (364 bytes) - added by MikeHansenMe 8 months ago.
Patch to remove the space
24914.2.diff (411 bytes) - added by SergeyBiryukov 8 months ago.

Download all attachments as: .zip

Change History (17)

comment:1 nacin9 months ago

  • Milestone changed from Awaiting Review to 3.8

comment:2 lancewillett9 months ago

Thanks, this was bugging me at WCSF Contributor Day, glad you've tracked it for us.

comment:3 obenland9 months ago

Have you tried it with MP6 activated?

comment:4 MikeHansenMe8 months ago

  • Cc mdhansen@… added

I noticed this as well. It does not happen with MP6 activated.

MikeHansenMe8 months ago

screenshot of space using chrome

MikeHansenMe8 months ago

Patch to remove the space

comment:5 MikeHansenMe8 months ago

This problem seems to affect more than just chrome. The patch I submitted fixes the problem for me on FF23 and Chromium28 on Ubuntu. This patch will need to be tested because the problem does not seem to affect all browser OS combos.

comment:6 obenland8 months ago

To explain the difference: In MP6 the toolbar is 4px taller than in pre-MP6 and Further was optimized for MP6.

comment:7 MikeHansenMe8 months ago

So it seems we need to wait this one out to see if MP6 makes it into core in 3.8. If not I think patching it would be a good idea so it will look right with default core toolbar.

comment:8 markoheijnen8 months ago

We can easily apply the patch and add this for now:
.admin-bar.mp6 #masthead.masthead-fixed { top:32px; }

comment:9 MikeHansenMe8 months ago

  • Keywords has-patch added; needs-patch removed

comment:10 follow-up: georgestephanis8 months ago

Honestly, as the plan is for them to ship together, I think we should just test them together. If MP6 drops (unlikely) then we can re-address it.

comment:11 in reply to: ↑ 10 nacin8 months ago

Replying to georgestephanis:

Honestly, as the plan is for them to ship together, I think we should just test them together. If MP6 drops (unlikely) then we can re-address it.

Replying to markoheijnen:

We can easily apply the patch and add this for now:
.admin-bar.mp6 #masthead.masthead-fixed { top:32px; }

I prefer the latter approach.

comment:12 georgestephanis8 months ago

That being said, we should account for the 12px gap between the 782px breakpoint where the MP6 header goes to 46px high http://plugins.trac.wordpress.org/browser/mp6/trunk/components/responsive/css/admin-bar.css#L15, and the 770px breakpoint where the nav stops being fixed http://core.trac.wordpress.org/browser/trunk/src/wp-content/themes/twentyfourteen/js/theme.js#L95

Version 0, edited 8 months ago by georgestephanis (next)

comment:13 obenland8 months ago

If we're backwards compatible with 3.6 (I assume we are) we'd have to accommodate for both toolbars, MP6 and non-MP6.

SergeyBiryukov8 months ago

comment:14 lancewillett8 months ago

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

In 25063:

Twenty Fourteen: ensure toolbar doesn't have a gap at the top when you start scrolling, including back compat for 3.6. Props MikeHansenMe and SergeyBiryukov, fixes #24914.

Note: See TracTickets for help on using tickets.