WordPress.org

Make WordPress Core

Opened 2 years ago

Closed 2 years ago

#20713 closed defect (bug) (fixed)

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

Reported by: johnbillion Owned by: koopersmith
Milestone: 3.4 Priority: normal
Severity: normal Version: 3.4
Component: Appearance 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 2 years ago.

Download all attachments as: .zip

Change History (4)

ocean902 years ago

comment:1 ocean902 years ago

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

comment:2 nacin2 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?

comment:3 koopersmith2 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.