Revert r42514 for now "(bug 16073) * Use onclick handler for expand/collapse in enhan...
authorBrion Vibber <brion@users.mediawiki.org>
Sat, 25 Oct 2008 00:10:07 +0000 (00:10 +0000)
committerBrion Vibber <brion@users.mediawiki.org>
Sat, 25 Oct 2008 00:10:07 +0000 (00:10 +0000)
commit846a61ae49d95f863f2d85cf49534ca93917f4f2
tree5b8342144f2696e58cabd596572757e6b5a8d8ac
parent27b3decd8160952b53028c44d2ef7fa7e8d05ba8
Revert r42514 for now "(bug 16073) * Use onclick handler for expand/collapse in enhanced recentchanges."

A few issues I noted:
* The expand/contract icons no longer have regular link behavior (eg no hand icon, can't be reached by keyboard tabbing)
* It looks like the stuff-to-be-hidden doesn't get hidden until after the </body>, which feels a little sketchy to me. On long lists and slow connections you may see odd behavior with the items being shown expanded, then suddenly hiding when things reach the end. Adding the style immediately in the JS instead of waiting for the body load completion should avoid that
* mw-rc-jshidden class seems to be applied to things that shouldn't have it sincec they are explicitly given display:none?
* Instead of style='display:none' etc, consider using clear classes for expanded and hidden modes, then switch the classes instead of the styles in the JS
RELEASE-NOTES
includes/ChangesList.php
includes/DefaultSettings.php
skins/common/enhancedchanges.js [deleted file]
skins/common/wikibits.js