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

Refactor ScrollBar not to use iframes #39

Closed
kyoshino opened this issue May 2, 2016 · 1 comment
Closed

Refactor ScrollBar not to use iframes #39

kyoshino opened this issue May 2, 2016 · 1 comment
Assignees
@kyoshino
Labels
compatibility performance widgets

Comments

@kyoshino
Copy link
Member

kyoshino commented May 2, 2016

Using a bunch of <iframe>s is not e10s-friendly, and the MozScrolledAreaChanged event is Gecko-specific. Rewrite the resize detector using requestAnimationFrame as explained in the resize doc. Blocks bzdeck/bzdeck#368.
@kyoshino kyoshino self-assigned this May 2, 2016
kyoshino added a commit that referenced this issue May 2, 2016
@kyoshino
Fix infinite loop on ComboBox, part of #39
68f7527 
Use clientHeight for the ScrollBar, instead of offsetHeight
kyoshino added a commit that referenced this issue May 2, 2016
@kyoshino
Improve ComboBox performance by manually updating scrollbar, part of #39
6cabc1e
@kyoshino kyoshino mentioned this issue May 3, 2016
Open
@kyoshino
Copy link
Member Author

kyoshino commented May 7, 2016 •
edited
Loading

Performance issue: requestAnimationFrame should not be used by each ScrollBar instance.

@kyoshino kyoshino reopened this May 7, 2016
kyoshino added a commit that referenced this issue May 8, 2016
@kyoshino
Improve ScrollBar performance, part of #39
41652aa
@kyoshino kyoshino closed this as completed May 8, 2016
@kyoshino kyoshino mentioned this issue May 28, 2016
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@kyoshino kyoshino

Labels
compatibility performance widgets
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@kyoshino