For example, the overlay will display fine at http://dev.diversifiedgreensolutions.com. It will also display fine at http://dev.diversifiedgreensolutions.com/our-services. Any page under 'Our Services', however, will not display the overlay (http://dev.diversifiedgreensolutions.com/our-services/total-waste-manage...).

I'm afraid of IE6 users clicking links that lead directly into these pages.

FYI. We are not optimizing for IE6, thus the reason for using this module. Most divs have been hidden so that nothing but the background appears with the overlay above that.

Thanks!
Kirk

CommentFileSizeAuthor
#5 noie6-js_path-1102902-5.patch781 bytesaltrugon
#3 noie6-js_path-1102902-3.patch700 byteshgmartini

Comments

altrugon’s picture

altrugon commented

Hello Kirk,

I tracked down the problem to a missing leading slash in the path to the noie6.js file. The issue is now fixed on the dev release and also there is a new stable release.

Thanks for using No IE6 module.

kirkofalltrades’s picture

kirkofalltrades commented

Thank you!

hgmartini’s picture

hgmartini commented
Status: Active » Needs review
StatusFileSize
new noie6-js_path-1102902-3.patch700 bytes

I think the change in comment 1 (if it refers to commit e2cac07fa2abe9c3f82aa452570a3816c1e8c3c7) broke the module for all pages, when the site is not in the server's root: the browser won't be able to load noie6.js because of a wrong path; the request will be /sites/all/modules/noie6/noie6.js, when it should be /{site-path}/sites/all/modules/noie6/noie6.js. It worked before because the path was relative, so the file could be found when the page was at the root of the site.

The original problem and this new one should be solved using file_create_url() to create the full path to the file (there might be another, "more correct" function to do this, I'm a newbie to Drupal coding).

I tested the attached patch against 7.x-1.0, but it should work with branch 7.x-2.x too; I think there are no changes in noie6.module between the two. Using this patch, the module worked for me in a site that's aliased to a subdirectory of the server's root, for pages of level 1 and deeper.

altrugon’s picture

altrugon commented

I'm a little bit tide with time right now to take a look at it. If some body else experience the same problem and can verify the patch work please let me know and I will do my best to apply it and create a new version for the module.

I'm really sorry for the inconveniences.

altrugon’s picture

altrugon commented
StatusFileSize
new noie6-js_path-1102902-5.patch781 bytes

Sorry for the late reply hgmartini.

Please try the patch attached to this comment against branch 7.x-2.x and let me know if it works. I think using $base_path the problem will get solved.

Cheers.

hgmartini’s picture

hgmartini commented

Sorry for the late reply too.

I just tried your patch and it works, at least for my setup (the site lives in a subdirectory of the server: http://{server}/{drupal_site}/{possible_subpath/}page.) Apparently $base_path does solve the problem.

Thank you!

altrugon’s picture

altrugon commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.