When juqery_update module is enabled, noie6 message don't displayed. I guess it's because jquery_update module rewrite $variables['scripts'] contents:
./jquery_update/jquery_update.module:84: $variables['scripts'] = drupal_get_js('header', $scripts);
Code to display message is not added trought drupal_add_js so it do not returned by drupal_get_js().
In any case, noie6 message is displayed if jquery_update module is switched off.

CommentFileSizeAuthor
#7 noie6_head.patch1018 bytesstackpr

Comments

altrugon’s picture

altrugon commented

I haven't experienced any problem with jquery_update module, can you tell me which version are you using?

I'll try to take a look to this issue as soon as I can.

suvisor’s picture

suvisor commented

My jquery_update version is 6.x-2.0-alpha1.

altrugon’s picture

altrugon commented
Status: Active » Closed (won't fix)

You are trying an alpha version, please try the recommended release.

If this problem continues one the alpha version get set as recommended then I will take a look at it.

4ud’s picture

4ud commented

It doesn't work also with jquery_update 6.x-2.x-dev (2010-Sep-09). But the Fancybox modul need this version of jquery_update.
Anyone has an idea to fix it? I need help.

Thanks, 4ud

altrugon’s picture

altrugon commented
Status: Closed (won't fix) » Postponed (maintainer needs more info)

I have notified the jquery_update team about this issue, waiting for their reply.

4ud’s picture

4ud commented

Thank you altrugon!
Would be much appreciated if they can help very soon ...

Hasta pronto, 4ud

stackpr’s picture

stackpr commented
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new noie6_head.patch1018 bytes

bundlecache and any other module that manipulates $scripts will probably break this too. The attached patch appears to work. The comment-based auto-detection is moved to the $head variable, and the JS is loaded at the end of the script block IFF the conditional comment JS is executed. It is slightly more complex, but it should be compatible with most modules, and it should not impact browser compatibility (since it already requires JS).

makkon’s picture

makkon commented
Version: 6.x-1.1 » 6.x-1.3

JQUpdate 6.x-2.0-alpha1 tar.gz (81.69 KB) | zip (85.94 KB) 2010-Apr-23
+
No IE6 Version:6.x-1.1
=
not working

altrugon’s picture

altrugon commented
Status: Needs review » Closed (works as designed)

Unfortunately makkon this issue can't be fixed until #965680: jquery_update_preprocess_page() overwrite $variables['scripts'] issue get solved.