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

.show()/.hide(): these methods may have performance issues #816

Closed
timmywil opened this issue Oct 16, 2015 · 1 comment
Closed

.show()/.hide(): these methods may have performance issues #816

timmywil opened this issue Oct 16, 2015 · 1 comment
Assignees
@AurelioDeRosa

Comments

@timmywil
Copy link
Member

We've resigned ourselves to reverting the current behavior, which means that we won't be able to get around the performance issues. Specifically, we have to retrieve computed display values, which is especially slow in webkit and blink.

Something like...

Note: Usage of these methods may cause performance issues, especially when used on many elements. Use performance testing tools to determine whether these methods are causing a problem.
@timmywil
Copy link
Member Author

Also add a note saying that show/hide can cause problems with responsive layouts, if the display value differs at different viewport sizes.

@AurelioDeRosa AurelioDeRosa self-assigned this Dec 14, 2015
AurelioDeRosa added a commit to AurelioDeRosa/api.jquery.com that referenced this issue Dec 16, 2015
@AurelioDeRosa
Added performance warning to show and hide
263b5e6 
Fixes jquerygh-816
@AurelioDeRosa AurelioDeRosa mentioned this issue Dec 16, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AurelioDeRosa AurelioDeRosa

Labels
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@timmywil @AurelioDeRosa