Posted by fernando iglesias on July 18, 2011 at 5:59pm
4 followers
| Project: | Quiz |
| Version: | 6.x-4.1 |
| Component: | User interface |
| Category: | support request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (won't fix) |
Issue Summary
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.
Comments
#1
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.
#2
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!
#3
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.)
#4
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.
#5
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.
#6
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.