Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
It looks around 55-60% width of the content area. It should be like 15% width as in other browsers.
Comment | File | Size | Author |
---|---|---|---|
#2 | poll-screenshot.png | 5.78 KB | Rowanw |
#1 | poll_column_width.patch | 459 bytes | jrabeemer |
poll-screenshot.jpg | 74.41 KB | jrabeemer |
Comments
Comment #1
jrabeemer CreditAttribution: jrabeemer commentedHere's a fix that fixes the width of the 'Vote Count' column in polls module. IE7 wasn't respecting the size attribute of the input field and fields get cut off when widths are applied. This patch sets a consistent size.
Tested and works consistently in IE7, Safari 3.0.4 Windows, FF2, Opera 9.5
Comment #2
Rowanw CreditAttribution: Rowanw commentedYour screenshot doesn't really demonstrate the problem clearly, I expected to see it exactly as IE7 rendered it. It took me a while to notice what you'd done.
I'm not sold on the patch though, the size of the 'choice' input field is 60, while the size of the 'count' input field is 5. IE is clearly misbehaving. I don't agree with adding more core CSS to cater for one of IE's trivial quirks.
Comment #3
jrabeemer CreditAttribution: jrabeemer commentedI hear what you're saying. In a perfect world, we'd rip out all the IE css out of style.css and fix-ie.css from the Garland theme.
Comment #4
BrightLoudNoise CreditAttribution: BrightLoudNoise commentedTested successfully in IE7 and IE6.
It handles the quirk at the module css level, which is far better than having to fix it in every theme in my opinion.
If down the road we implement an IE quirks CSS file for core, we could move it then.
Comment #5
jrabeemer CreditAttribution: jrabeemer commentedIE6 and IE7 are our biggest audience.
Comment #6
Gábor HojtsyWho tested in other browsers? Firefox, Safari, Opera?
Comment #7
BrightLoudNoise CreditAttribution: BrightLoudNoise commentedSorry about that Gabor, you are quite correct in pointing out the incompletness of my earlier test.
Tested successfully in Safari 3.0.4, Opera 9.25 and Firefox 2.0.0.11.
Comment #8
jrabeemer CreditAttribution: jrabeemer commentedThanks for the 2nd confirmation.
Comment #9
Gábor HojtsyCommitted, thanks.
Comment #10
Anonymous (not verified) CreditAttribution: Anonymous commentedAutomatically closed -- issue fixed for two weeks with no activity.