Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 UTC on 18 March 2024, to get $100 off your ticket.
This patch disables sticky table headers for IE < 7. Nobody wants to fix them and they are horribly broken.
Comment | File | Size | Author |
---|---|---|---|
#3 | tableheader-exclude-ie6_1.patch | 661 bytes | dvessel |
tableheader-exclude-ie6.patch | 592 bytes | kkaefer | |
Comments
Comment #1
dvessel CreditAttribution: dvessel commentedTested in IE6 and no more tables headers. This is a UI enhancement so disabling for IE6 I think is perfectly acceptable.
Also tested in:
Safari 2, Opera 9, FF2
Comment #2
Gábor HojtsyAdd a comment about why we do this. Also I guess our coding style guidelines stand in JS, so you need braces and the return on a new line.
Comment #3
dvessel CreditAttribution: dvessel commentedPer your request..
Comment #4
Gábor HojtsyOK, this version committed. I agree that this is not the most ideal approach, but we are not having better now, and this needs to be fixed for IE < 7 users.
Comment #5
kkaefer CreditAttribution: kkaefer commentedIf anybody is willing to fix this for IE6, he/she can go ahead. I'm not willing to invest time in fixing such things for overly broken and outdated browsers.
Gábor, where are the coding standards for JS?
Comment #6
Gábor HojtsyI don't think there are JS coding standards written.
Comment #7
(not verified) CreditAttribution: commented