#7257 closed enhancement (fixed)
Core: optimize :tabbable
| Reported by: | adambaratz | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | 1.9.0 |
| Component: | ui.core | Version: | 1.8.11 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
The :tabbable selector is dependent on the :focusable selector. Both pull the tabindex attribute on each element they check. Some milliseconds can be shaved by reusing this value. It's a slight gain, but since :tabbable is run during Dialog initialization, complex dialogs can have real gains from this.
Change History (4)
comment:1 Changed 12 years ago by
comment:2 Changed 12 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Core: reuse tabindex value. Fixed #7257 - optimize :tabbable.
Changeset: fe3b36b8efcabf3d1f1d60523cae87c69a882f27
comment:3 Changed 12 years ago by
Core: reuse tabindex value. Fixed #7257 - optimize :tabbable. (cherry picked from commit fe3b36b8efcabf3d1f1d60523cae87c69a882f27)
Changeset: 96f6b1a7db1e3bc74de66153cf9be80152e04430
comment:4 Changed 12 years ago by
This still performs very poorly (eg Firefox 5 on a new dialog with 300 elements takes around 5 seconds).
Note: See
TracTickets for help on using
tickets.
https://github.com/adambaratz/jquery-ui/commit/a847d8644a5102939e13133e5d8601e856d814a9