Issue status update for http://drupal.org/node/29075 Post a follow up: http://drupal.org/project/comments/add/29075 Project: Drupal Version: cvs Component: other Category: bug reports Priority: critical Assigned to: fago Reported by: fago Updated by: fago -Status: patch (code needs work) +Status: patch (code needs review) Attachment: http://drupal.org/files/issues/ie-fieldset-fix.patch (696 bytes) i 've followed dries hints and created a new patch. this one includes both above patches. fago Previous comments: ------------------------------------------------------------------------ Wed, 17 Aug 2005 20:04:21 +0000 : fago Attachment: http://drupal.org/files/issues/arrow-ie5.patch (361 bytes) ie 5.0 displays the menu-collapsed/expanded.png images behind the legend link text. fix attached ------------------------------------------------------------------------ Wed, 17 Aug 2005 20:22:58 +0000 : Thox 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. ------------------------------------------------------------------------ Wed, 17 Aug 2005 20:56:44 +0000 : stefan nagtegaal 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.. ------------------------------------------------------------------------ Wed, 17 Aug 2005 20:59:35 +0000 : fago Attachment: http://drupal.org/files/issues/ie-table-fix.patch (495 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 ------------------------------------------------------------------------ Thu, 18 Aug 2005 00:33:11 +0000 : gordon +1 This flex my problems with the tables not expanding correctly under IE. Please apply. ------------------------------------------------------------------------ Thu, 18 Aug 2005 05:18:02 +0000 : Dries 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.