Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

fix(virtualRepeat): Recover from scroll events that occur when hidden #6389

Closed
wants to merge 1 commit into from

Conversation

kseamon
Copy link
Contributor

@kseamon kseamon commented Dec 18, 2015

Fixes #5448.

@kseamon
Copy link
Contributor Author

kseamon commented Dec 18, 2015

@@ -197,6 +197,7 @@ VirtualRepeatContainerController.prototype.updateSize = function() {
? this.$element[0].clientWidth
: this.$element[0].clientHeight;

this.handleScroll_();
Copy link
Member

Choose a reason for hiding this comment

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

Maybe add a comment as to why you're calling the scroll handler here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@kseamon kseamon force-pushed the autocomplete-white-whale branch from b98e870 to cff2654 Compare December 18, 2015 20:14
@jelbourn
Copy link
Member

LGTM

@jelbourn jelbourn added pr: merge ready This PR is ready for a caretaker to review g3: pr This PR was posted by an internal or external product team. labels Dec 18, 2015
@kseamon kseamon closed this in bbca34f Dec 21, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
g3: pr This PR was posted by an internal or external product team. pr: merge ready This PR is ready for a caretaker to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants