Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
IE7 only.
Before:
After:
Comment | File | Size | Author |
---|---|---|---|
autocomplet_ie7.patch | 341 bytes | droplet | |
IE7 only.
Before:
After:
Comment | File | Size | Author |
---|---|---|---|
autocomplet_ie7.patch | 341 bytes | droplet | |
Comments
Comment #1
JamesK CreditAttribution: JamesK commentedLooks good. Nice catch droplet.
Comment #2
David_Rothstein CreditAttribution: David_Rothstein commentedThanks for the patch and screenshots.
I think before we fix this in system.base.css, it would be good to verify that this bug is actually theme-independent (i.e., that it's reproducible even in the Stark theme).
Also, as far as I know the "zoom" property is invalid/proprietary CSS... which I'm assuming we want to avoid (especially in system.base.css) if at all possible?
Given the above, I think this patch could probably use another review, ideally by someone with a stronger CSS background than me :) I'm moving it to a location in the issue queue where that is more likely to happen.
Comment #3
droplet CreditAttribution: droplet commented- Bug exist in Stark theme ?
Yes
- W3C invalid CSS hack already used in system sheets
Comment #4
David_Rothstein CreditAttribution: David_Rothstein commentedOK, thanks.
Tell you what, let's move this to back to RTBC, but I won't commit it just yet. If it stays RTBC in the CSS section of the issue queue for a little while with no one coming along and saying anything's wrong with it, I'll assume it's OK to commit then :)
Comment #5
kristiaanvandeneyndeThere is indeed some invalid CSS in system.base.css already (clearfix) and the zoom property, when set to 1, is completely harmless in non-IE browsers. Still RTBC as far as I'm concerned.
If you are concerned with the IE-proprietary properties in system.base.css, you could always open a new issue asking to move all of them to an IE-specific stylesheet (in D7, not D8)?
Comment #6
droplet CreditAttribution: droplet commentedBasically, there are not all of IE problems. -prefix are not W3C Validated code too.
We can remove more non-standard code out of CORE modules:
#1422614: Drop Firefox 3.6 support in Drupal core
Comment #7
kristiaanvandeneyndeTrue, although vendor prefixes tend to be used mostly alongside CSS3 prefixes, e.g.:
-moz-border-radius: ...;
-webkit-border-radius: ...;
-border-radius: ...;
Regardless, you have a point...
Comment #8
David_Rothstein CreditAttribution: David_Rothstein commentedOK, it's been here a while and no one has complained, and one person even gave it a +1, so...
Committed to 7.x - thanks! http://drupalcode.org/project/drupal.git/commit/eb35c10
Comment #9
David_Rothstein CreditAttribution: David_Rothstein commentedForgot to mark it "fixed".
By the way, does this need backport to 6.x also?
Comment #10
David_Rothstein CreditAttribution: David_Rothstein commentedThe CSS looks pretty similar in Drupal 6 and doesn't have the zoom property, so I'm going to assume for now that this exists there also.
Comment #11
droplet CreditAttribution: droplet commented