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

fix(util): restore scrolling after test executed. #8206

Closed

Conversation

devversion
Copy link
Member

  • The current test, was not restoring the scroll mask, which means, that we're not able to remove the scrollbars on the karma site.

This broke some positioning tests for feat/panel.

cc. @ErinCoughlan

* The current test, was not restoring the scroll mask, which means, that we're not able to remove the scrollbars on the karma site.

This broke some positioning tests for `feat/panel`.
@ErinCoughlan
Copy link
Contributor

LGTM

@devversion devversion added the needs: review This PR is waiting on review from the team label Apr 25, 2016
@ThomasBurleson ThomasBurleson added pr: merge ready This PR is ready for a caretaker to review and removed needs: review This PR is waiting on review from the team labels Apr 25, 2016
@ThomasBurleson ThomasBurleson added this to the 1.1.0 milestone Apr 25, 2016
@devversion devversion deleted the fix/util-scrolling-test branch April 25, 2016 19:32
jelbourn pushed a commit that referenced this pull request Apr 28, 2016
* The current test, was not restoring the scroll mask, which means, that we're not able to remove the scrollbars on the karma site.

This broke some positioning tests for `feat/panel`.

Closes #8206
ThomasBurleson pushed a commit that referenced this pull request May 17, 2016
* The current test, was not restoring the scroll mask, which means, that we're not able to remove the scrollbars on the karma site.

This broke some positioning tests for `feat/panel`.

Closes #8206
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
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.

3 participants