Skip to content
This repository has been archived by the owner on Nov 17, 2020. It is now read-only.

Scroll to the page top when rendering a new template. #590

Merged
merged 2 commits into from
Jul 20, 2018
Merged

Scroll to the page top when rendering a new template. #590

merged 2 commits into from
Jul 20, 2018

Conversation

hairyhum
Copy link
Contributor

lukebakken
lukebakken approved these changes Jul 20, 2018
View reviewed changes
Copy link
Contributor

@lukebakken lukebakken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reproduced using PerfTest and 100 producers / consumers, and confirmed fix works great 👍

@lukebakken lukebakken added this to the 3.7.8 milestone Jul 20, 2018
@lukebakken lukebakken merged commit b2fa1ef into master Jul 20, 2018
@lukebakken lukebakken deleted the scroll-to-top-after-navigate branch July 20, 2018 14:05
lukebakken added a commit that referenced this pull request Jul 20, 2018
@lukebakken
Merge pull request #590 from rabbitmq/scroll-to-top-after-navigate
15abb69 
Scroll to the page top when rendering a new template.

(cherry picked from commit b2fa1ef)
@lukebakken
Copy link
Contributor

v3.7.x: 15abb69

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@lukebakken lukebakken lukebakken approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
3.7.8
Development

Successfully merging this pull request may close these issues.
2 participants
@hairyhum @lukebakken