Would it be possible to have a different message that appears for IE7 people so they can be (possibly even more gently) prompted to upgrade to IE8? (While still retaining the original IE6 functionality)
Great module by the way. :-)
Would it be possible to have a different message that appears for IE7 people so they can be (possibly even more gently) prompted to upgrade to IE8? (While still retaining the original IE6 functionality)
Great module by the way. :-)
Comments
Comment #1
jrglasgow commentedthis is a possibility, let me think on the best way for a bit
Comment #2
BigEd commentedmm I could do with this as well.
I have taken a look at the module and this should be pretty easy to implement as its based on the conditional statements.
Wouldnt it be just a case of tweaking it to recognise Ie7?
I can take a look on mine if you need but if you are working on this let me know.
Comment #3
marafa commentedping
Comment #4
fureigh commentedThe simplest way is to change this line in ie6update.js:
to this:
Adding an admin setting to allow this to be switched on or off would be preferable, which is why I haven't attached a patch.
Comment #5
jmbouvier commentedThis is my exact problem too - this works well in ie6 but doesn't appear in ie7 when I've cut and pasted #4.
Any ideas?
Comment #6
danlinn commentedI might be a little late to the party here, but in order to get it to show in IE7, you also need to modify ie6update.module and change line 55 to [if lte IE 7] instead of [if IE 6]
Party on.
Comment #7
chiappa commentedThanks but in this way it only shows for IE7 not IE6. How can I have it working for both?
Comment #8
lucascaro commentedThat code should work for ie 5, 6 and 7, are you sure you copied it right?
you could also try the patch in:
http://drupal.org/node/843926
cheers!
Comment #9
iamEAP commentedRe-opening this. If anyone still on this thread is interested enough, I think a configurable conditional stylesheet value is the way to go.
One question I have is: how similar are the message bars at the top of IE6 vs. IE7 vs. IE8 stylistically?
Is it worth re-styling them?
Comment #10
iamEAP commentedAccording to the IE Conditional Comments documentation, you can use a very basic OR logical operator in conditions. So, for maximum flexibility, I've just set up the config to be checkboxes. By default, only IE6 is selected, but you can select any combination of them, producing a conditional string like the following:
<!--[if (IE 6)|(IE 7)]>As I noted earlier, I'm not sure if the message bar is consistent stylistically across different versions of IE, so it may appear bizarre in versions IE greater than 6.
Committed to 7.x branch: b7a3540. If anyone still following this is active and wants to test it out, by all means!
Comment #11
iamEAP commentedCommitted the same to 6.x branch: 6a71bc3
Comment #13
a.ross commentedIt may be helpful to add this info to the project description. I didn't know I could use this for IE7 (and up) until I ran into this issue.
Comment #14
iamEAP commentedThe info was on the description page, but I tried highlighting it a bit more.
In the future, please consider opening a new issue rather than re-opening an old one. Nevertheless, thanks for the feedback.