https://bugzilla.wikimedia.org/show_bug.cgi?id=34876

--- Comment #6 from Lupo <[email protected]> 2012-03-08 10:02:54 UTC ---
Created attachment 10198
  --> https://bugzilla.wikimedia.org/attachment.cgi?id=10198
Patch for further improvement after r112457

The .addClass ( 'mw-collapsible mw-made-collapsible' ) also is rather
expensive, probably because it causes reflows of pretty much the whole page.

I see that in r112457 you had already eliminated the need for the
mw-made-collapsible class in CSS. Not using a class for this marker but storing
it instead as data again helps avoid page reflows and brings even more speedup.

The patch implements this (plus has an optimized getRows() function to use
instead of .children( 'tbody' ).children ( 'tr:first' ), which incurs expensive
Sizzle invocations).

The patch does require r112457, though. As long as there's CSS that relies on
that mw-made-collapsible class to display the toggle arrows, you can't avoid
setting that class.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

_______________________________________________
Wikibugs-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to