This patch disables sticky table headers for IE < 7. Nobody wants to fix them and they are horribly broken.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dvessel’s picture

Status: Needs review » Reviewed & tested by the community

Tested 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

Gábor Hojtsy’s picture

Status: Reviewed & tested by the community » Needs work

Add 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.

dvessel’s picture

Status: Needs work » Reviewed & tested by the community
FileSize
661 bytes

Per your request..

Gábor Hojtsy’s picture

Status: Reviewed & tested by the community » Fixed

OK, 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.

kkaefer’s picture

If 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?

Gábor Hojtsy’s picture

I don't think there are JS coding standards written.

Anonymous’s picture

Status: Fixed » Closed (fixed)