Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Performance: background event listeners #7049

Closed
manucorporat opened this issue Jun 25, 2016 · 1 comment
Closed

Performance: background event listeners #7049

manucorporat opened this issue Jun 25, 2016 · 1 comment
Assignees
@manucorporat

Comments

@manucorporat
Copy link
Contributor

manucorporat commented Jun 25, 2016 •
edited

Components like Sliding Items currently register a fullscreen touchmove/mousemove event listener that can cause jank in slow devices (and less noticeably in fast ones) while scrolling.

screen shot 2016-06-25 at 02 06 55

After some experimentation, I was able to prevent these slow highly nested events by using the new UIEventManager.

screen shot 2016-06-25 at 02 11 23

I will continue investigating if we can improve the performance of Swipe to go back, and swipe to show the menu.
@Ionitron Ionitron added the menus label Jun 25, 2016
@manucorporat manucorporat mentioned this issue Jun 27, 2016
Closed
12 tasks
@jgw96 jgw96 added the v2 label Jun 27, 2016
@manucorporat manucorporat self-assigned this Jul 2, 2016
manucorporat added a commit to manucorporat/ionic that referenced this issue Jul 3, 2016
@manucorporat
fix(sliding): much better UX + performance
28417e7 
- sliding should behave exactly like a native one
- much better performance

references ionic-team#7049
references ionic-team#7116
closes ionic-team#6913
closes ionic-team#6958
This was referenced Jul 3, 2016
Closed
Closed
manucorporat added a commit to manucorporat/ionic that referenced this issue Jul 3, 2016
@manucorporat
fix(sliding): much better UX + performance
e1a97b2 
- sliding should behave exactly like a native one
- much better performance

references ionic-team#7049
references ionic-team#7116
closes ionic-team#6913
closes ionic-team#6958
manucorporat added a commit to manucorporat/ionic that referenced this issue Jul 12, 2016
@manucorporat
fix(sliding): much better UX + performance
d6f62bc 
- sliding should behave exactly like a native one
- much better performance

references ionic-team#7049
references ionic-team#7116
closes ionic-team#6913
closes ionic-team#6958
@manucorporat
Copy link
Contributor Author

Fixed! 5909fa4

@ionitron-bot ionitron-bot bot locked and limited conversation to collaborators Sep 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@manucorporat manucorporat

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@manucorporat @Ionitron @jgw96