Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#20713 closed defect (bug) (fixed)

Theme customizer: Scrolling down then navigating to another page jumps the subsequent page down

Reported by: johnbillion's profile johnbillion Owned by: koopersmith's profile koopersmith
Milestone: 3.4 Priority: normal
Severity: normal Version: 3.4
Component: Customize Keywords: has-patch commit
Focuses: Cc:

Description

Steps to reproduce:

  1. Open the theme customizer
  2. Scroll down the page and click a link

The subsequent page will scroll down the same distance as you were scrolled when you clicked the link instead of loading at the top.

Attachments (1)

20713.patch (565 bytes) - added by ocean90 12 years ago.

Download all attachments as: .zip

Change History (4)

@ocean90
12 years ago

#1 @ocean90
12 years ago

  • Component changed from Themes to Appearance
  • Keywords has-patch ux-feedback added
  • Milestone changed from Awaiting Review to 3.4

#2 @nacin
12 years ago

  • Keywords commit added; ux-feedback removed

Looks good. The scroll stuff is there to allow for page refreshes when a control is changed, but a click should definitely reset it to the top. Koop?

#3 @koopersmith
12 years ago

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

In [20833]:

Theme Customizer: Reset scroll to top when the preview url is changed. props ocean90, fixes #20713, see #19910.

Note: See TracTickets for help on using tickets.