Skip to content

fix resizing collisions caused by asynchronous nature of scope.$watch#33

Open
Merott wants to merge 1 commit intopatrickmarabeas:masterfrom
Merott:fix-collisions
Open

fix resizing collisions caused by asynchronous nature of scope.$watch#33
Merott wants to merge 1 commit intopatrickmarabeas:masterfrom
Merott:fix-collisions

Conversation

@Merott
Copy link
Copy Markdown
Contributor

@Merott Merott commented Jul 31, 2015

The resizing sometimes wouldn't work as expected because resize was being called at different times with different values, and not always the most recent would win. By implementing a short delay, and ensuring that previous calls to resize() are cancelled, the resizing can be done more reliably.

The resizing sometimes wouldn't work as expected because the resize function
was being called at different times with different values, and not always
the most recent would win. By implementing a short delay, and ensuring that
previous calls to resize() are cancelled, the resizing can be done more reliably
@duezerouno
Copy link
Copy Markdown

Fixes #54. Thank you @Merott

@LoganLehman
Copy link
Copy Markdown

@Merott @patrickmarabeas - Can we please get this merged?

@patrickmarabeas
Copy link
Copy Markdown
Owner

Integrated the changes; check v4.2.2 and see whether it works as expected.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants