Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#28898 closed defect (bug) (fixed)

Theme-install preview toolbars don't consistently load

Reported by: melchoyce Owned by: markjaquith
Priority: normal Milestone: 4.1
Component: Themes Version: 4.0
Severity: normal Keywords: has-patch
Cc: Focuses: administration

Description

The "add theme" preview toolbar doesn't consistently load for me. See it in action: ​https://cloudup.com/cHSv7TS9dPz

Not seeing anything in the console.

Attachments (1)

28898.patch​ (326 bytes ) - added by ocean90 12 years ago.

Download all attachments as: .zip

Change History (11)

#1 @celloexpressions
12 years ago

  • Keywords reporter-feedback added
  • Milestone Awaiting Review → 4.0
  • Summary Theme preview toolbars don't consistently load → Theme-install preview toolbars don't consistently load

Most likely a browser rendering issue, but something we may be able to fix. I can't reproduce in Chome 36 or Firefox 31 on Windows 8.1, even with navigating in and out of the previewer very quickly with the keyboard.

There's a good chance this was introduced in 4.0 with the changes from buttons to icons here; @melchoyce, can you try it on other browsers and in 3.9 and narrow down where the issue is? If it isn't a regression, we need to punt it.

This ticket was mentioned in IRC in #wordpress-dev by helen. ​View the logs.


12 years ago

#3 @adamsilverstein
12 years ago

Mel, I tried to reproduce the issue mentioned in this ticket without luck - although your screen-cast makes it look easy.

I tried in Chrome/Dev and Firefox. Any tricks to getting the bug to surface? what browser are you using?

#4 @helen
12 years ago

  • Milestone 4.0 → Awaiting Review

Punting for now, this may very well be a "worksforme" situation even though I know it's been happening to melchoyce for a while now.

#5 @ocean90
12 years ago

Seeing this now too in Chrome 37/OSX, see ​https://cloudup.com/iEQan6lsivD.

@ocean90
12 years ago

#6 @ocean90
12 years ago

With 28898.patch​ I see the header and footer again. But no idea what's going on here.

#7 @ocean90
12 years ago

Seems like an issue in combination with a Retina screen. On a Retina screen it's hidden, after moving the browser to the my second screen (non-Retina) header and footer are coming back.

#8 @adamsilverstein
12 years ago

  • Keywords has-patch added

verified the patch fixes the problem. i think removing overflow: auto; makes sense, not sure why its there. I didn't see any adverse side effects from the change and after the patch can no longer reproduce the issue.

looks good, nice catch!

#9 @johnbillion
12 years ago

  • Keywords reporter-feedback removed
  • Milestone Awaiting Review → 4.1

#10 @markjaquith
12 years ago

  • Owner set to markjaquith
  • Resolution → fixed
  • Status new → closed

In 30101:

Fix weird HiDPI screen rendering issue with theme install preview toolbar

fixes #28898
props ocean90

Note: See TracTickets for help on using tickets.