← Back to context Comment by baddox 14 years ago Since you're using jQuery, you might as well just use show() and hide() or remove(). 4 comments baddox Reply WickyNilliams 14 years ago or even better, use toggle(): jQuery("#mw-sopaOverlay, .noprint, #footer, #content").toggle(); th 14 years ago Also chaining: https://gist.github.com/1631368 baddox 14 years ago I think that looks and scans worse. I didn't count characters, but it also seems like more characters. I bet it performs worse too (not that performance matters in this case). artursapek 14 years ago And execute the first command just on $('div')
WickyNilliams 14 years ago or even better, use toggle(): jQuery("#mw-sopaOverlay, .noprint, #footer, #content").toggle();
th 14 years ago Also chaining: https://gist.github.com/1631368 baddox 14 years ago I think that looks and scans worse. I didn't count characters, but it also seems like more characters. I bet it performs worse too (not that performance matters in this case).
baddox 14 years ago I think that looks and scans worse. I didn't count characters, but it also seems like more characters. I bet it performs worse too (not that performance matters in this case).
or even better, use toggle():
Also chaining: https://gist.github.com/1631368
I think that looks and scans worse. I didn't count characters, but it also seems like more characters. I bet it performs worse too (not that performance matters in this case).
And execute the first command just on $('div')