ie 5.0 displays the menu-collapsed/expanded.png images behind the legend link text.
fix attached

CommentFileSizeAuthor
#7 ie-fieldset-fix.patch696 bytesfago
#4 ie-table-fix.patch495 bytesfago
arrow-ie5.patch361 bytesfago

Comments

fago’s picture

fago
Primary language German
Location Vienna
commented
Version: 4.6.0 »
Thox’s picture

Thox commented

I'd like to see your IE table fix in the patch too, it's much cleaner than any of the approaches I was trying.

I've tested the current patch and it works fine in IE5, IE6, FF, Opera, and an old copy of Konqueror. I'd like to know that the patch doesn't affect Safari before it gets applied.

Stefan Nagtegaal’s picture

Stefan Nagtegaal commented

I can confirm that it does not effect Safari, because the CSS is never applied to the document because the AJAX/JS doesn't work on Safari..

fago’s picture

fago
Primary language German
Location Vienna
commented
StatusFileSize
new ie-table-fix.patch495 bytes

ok, here it is :)
-> fixes tables in collapsed fieldsets for IE5 and 6

i've tested both patches in IE5.0, IE6, FF, opera7 and konqueror 3.4

gordon’s picture

gordon commented
Priority: Minor » Critical
Status: Needs review » Reviewed & tested by the community

+1 This flex my problems with the tables not expanding correctly under IE.

Please apply.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Needs work

I'd add a code comment to document why this is needed.

You can shorten that patch by merging it with the next block in the CSS file.

fago’s picture

fago
Primary language German
Location Vienna
commented
Status: Needs work » Needs review
StatusFileSize
new ie-fieldset-fix.patch696 bytes

i 've followed dries hints and created a new patch.
this one includes both above patches.

dries’s picture

dries commented
Status: Needs review » Fixed

Committed to HEAD.

curry’s picture

curry commented
Anonymous’s picture

(not verified) commented
Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)