Stray Javascript

[feldon23]

I raised this here but it got lost in the shuffle.


Notice the commented Javascript at the top.

I don't think you can nest Javascript like this:

<script type="text/javascript">
    <!--
        document.write('<script type="text/javascript" src="' + yuipath + '/animation/animation-min.js?v=4111c"></script>');
        var sidebar_align = 'right';
        var content_container_margin = parseInt('290px');
        var sidebar_width = parseInt('270px');
    //-->
    </script>

[Wayne Luke]

I don't see this. Do you still see it?

[Geni]

Visible for me in firefox 3.6.27, not visible in safari.

[Wayne Luke]

Most likely an issue in that version of Firefox. Maybe it doesn't support the construct. I can't see it in Firefox 10, IE 9, Chrome 17 or Safari 5 under Windows.

As far as I am aware Javascript however does allow the inclusion of other scripts from within code using the methods outlined in the headinclude template of vBulletin.

[Yves R.]

Which browser can you see this? Firefox 3.6 is no more supported.

[feldon23]

The JS seems a very odd construct to me.

Why is yuipath a javascript variable instead of a stylevar? As far as I can tell that's the only reason there's nested javascript there, just to set a variable to yuipath = 'clientscript/yui';

I'm even more puzzled that the javascript is commented out. Is that a IE7 workaround?

Which browser can you see this? Firefox 3.6 is no more supported.

Firefox 3.6.27. I guess we'll see how long they release updates, then I'll have to seriously look at Google Chrome. Yuck.

Firefox 4/5/6/7/8/9/10/11/12/whatever are buggy, constant freezing rubbish.

[feldon23]

Just updated to Firefox v.3.6.28 (released March 13th, 2012). Neener.