Tech

Al

I've been struggling to make sure my syntax is correct to detect Safari and to load the appropriate stylesheet, but I'm not having any luck. I'm not much - if at all - a js programmer, so that's an issue :-\ Here's what I've got:

:: Code ::

There must be an error here...

Thanks for looking,
a!

jeffd · Posted: Mar 24, 06, 17:16

Hi Al, although you've created the function browser_css(), you haven't called it.

So after the } you have to add this line:

:: Code ::

browser_css();

which would look like this:

:: Code ::

.....
}
else {
d.write('<link rel = "stylesheet" type = "text\/css" href = "css/global.css" />');
}
}
browser_css();

This code would have to occur in the <head> tag by the way, since that's where the link stuff is located.

If I were doing this, I'd just put the browser_css() function in the browser_detection.js library file, then just call the whole thing with the function browser_css();, which would change your code to this:

:: Code ::

and that's all. Any repeated code in general should always be pulled off the page and put into library files, for easier maintainance.

Al · Posted: Mar 27, 06, 10:17

Hi Jeff,

It worked like a charm. Thanks for answering my js 101 question.

a!

Tech

Tech

patterns.com

tech forums