What's wrong with my codes?I want to load the js when the browser is great than IE8 or is not IE?

Question

0.00/5 (No votes)

See more:

JavaScript

 <!--[if (gt IE 8)|(!IE)]>
<script src="/FileSaver.min.js"></script>
<script src="/underscroe.js"></script>
<![endif]-->

I write the code like this and use firefox to debug it does not work,but When I write the codes directly,it works well.

JavaScript

<script src="/FileSaver.min.js"></script>
<script src="/underscroe.js"></script>

Posted 17-Jun-13 4:44am

PEIYANGXINQU

Add a Solution

Comments

Prasad Khandekar 17-Jun-13 11:27am

The conditional comments syntax is supported by IE only. For cross browser use javascript code something like shown below.

<script type="text/javascript">
_rwebkit = /(webkit)[ \/]([\w.]+)/;
_ropera = /(opera)(?:.*version)?[ \/]([\w.]+)/;
_rmsie = /(msie) ([\w.]+)/;
_rmozilla = /(mozilla)(?:.*? rv:([\w.]+))?/;

var brow = browserMatch(navigator.appVersion);
if (brow.browser !== 'msie' || (brow.browser === 'msie' && brow.version > 8)) {
var script = document.createElement("script");
script.setAttribute("src", 'FileSaver.min.js');
document.getElementsByTagName("head")[0].appendChild(script);

script = document.createElement("script");
script.setAttribute("src", 'underscroe.js');
document.getElementsByTagName("head")[0].appendChild(script);
}

// Taken from JQuery 1.7.1
function browserMatch(strUa) {
strUa = strUa.toLowerCase();
var match = _rwebkit.exec( strUa ) ||
_ropera.exec( strUa ) ||
_rmsie.exec( strUa ) ||
ua.indexOf("compatible") < 0 && _rmozilla.exec( strUa ) ||
[];

return { browser: match[1] || "", version: match[2] || "0" };
}
</script>

Zafar Sultan 17-Jun-13 11:58am

Hi Prasad.
Sometimes there is a need to fix something such as css or appearance of certain elements when the browser is IE(for obvious reasons). The conditional comments are really useful as they are supported by IE only. Its kind of shortcut to execute commands only in case of IE. I think that was what the OP wanted to achieve. Anyways your solution is perfect for situations where a user want to perform cross browser tasks. You should have posted it as a solution. I wish I could give a +5 for that.

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Zafar Sultan · Accepted Answer · 2013-06-17T05:46:00

Solution 1

Try this:

HTML

<!--[if gt IE 8]><!-->
Load your scripts here. But first make sure this message is displayed if your browser is IE8 or not IE
<!--<![endif]-->

Also keep in mind, according to Microsoft Document[^] these statements will not work in IE10

Posted 17-Jun-13 5:46am

Zafar Sultan

Updated 17-Jun-13 5:48am

v2

Comments

PEIYANGXINQU 17-Jun-13 13:15pm

Thank you, when user use IE10 to view my web,you say these statements will not work ,but in fact in IE10 my scripts also work,I think because of '<!-->'.