Lines 58-64 of quiz.css:

.quiz-question-browser-row.odd {
  background-color: #e7e7e7;
}

.quiz-question-browser-row.even {
  background-color: #fbfbfb;
}

Since Internet Explorer 6 doesn't properly handle multiple classes in a single selector, the browser chops off the starting classes and reads it like this:

.odd {
  background-color: #e7e7e7;
}

.even {
  background-color: #fbfbfb;
}

Because of this it can break the styling of most websites for IE6 users.

CommentFileSizeAuthor
#5 quiz_ie6_compatibility-1222356-5.patch1.68 KBfernando iglesias

Comments

sivaji_ganesh_jojodae’s picture

sivaji_ganesh_jojodae commented
Category: bug » support

Sorry Iglesias, I see this as a bug in IE6 perhaps quiz couldn't do anything about it. IE6 is almost abandoned by many web projects these days including Google. Drupal core would do the same in future see #308865: Drop IE6 support in Drupal core. Unless falcon finds a fix I suggest to mark this issue as won't fix.

vegardjo’s picture

vegardjo commented

I've just looked at this a second but you *might* be able to solve this by using: 

fieldset#edit-browser tr.odd {
  background-color: #e7e7e7;
}

fieldset#edit-browser tr.even {
  background-color: #fbfbfb;
}

..instead.

If Fernando or someone could check it in IE6 and also see that it has no side effects in other browsers it would be an easy fix. However, not supporting IE6 at all anymore would be perfectly fine with me!

falcon’s picture

falcon commented

I don't want to waste a second on IE6. Sorry. (If sivaji want to test the patch from vegardjo it is fine with me off-course.)

fernando iglesias’s picture

fernando iglesias commented

Thanks for the replies, everyone. #2 breaks the styling on the question browser since there doesn't seem to be an edit-browser id on the parent fieldset (at least not in 6.x-4.1) so the rules aren't actually applied. I'll keep playing around with it, if I figure out something that works I'll upload a patch.

fernando iglesias’s picture

fernando iglesias commented
Status: Active » Needs review
StatusFileSize
new quiz_ie6_compatibility-1222356-5.patch1.68 KB

Here's a patch, seems to work for me. I went with the suggestion in #2 and added the id to that parent fieldset and then adjusted the style so it continues to work in current browsers but also works in IE6.

sivaji_ganesh_jojodae’s picture

sivaji_ganesh_jojodae commented
Status: Needs review » Closed (won't fix)

Fernando, thanks for your patch but sorry I'm not in favor of supporting an abandoned version of IE. I will update the project page to reflect this with a link to this issue to credit your contribution.