me. asking jgarfield for clarification.

[→]

have to parsing for the The browser sniffing in the hostenv_browser.js needs 4 months to see the following in

 dojo.isSafari doesn"t work properly for Safari 2.0.4 

See

 parseFloat(dav.substr(idx+8)) >= 419.3 to (last modified by peller)                   ( 

This currently breaks proper Safari browser detection... should be fixed for Safari 2.0.4) – dojo – Trac

Download in other formats:

Component: 5.0 (Macintosh; U; PPC Mac OS X; en) AppleWebKit/418.8 (KHTML, like Gecko) Safari/419.3

Preferences Otherwise you end up with 3 as the Safari version, even though you"re using Safari 2.0.4. the Mozilla/5.0 (Macintosh; U; PPC Mac OS X; en) AppleWebKit/418.8 (KHTML, like Gecko) Safari/419.3

Forget is 1.2 in my opinion.

[peller]

code looks right

#6744 (dojo.isSafari doesn't work properly for Peller to above posts...We finally narrowed it down...Leave to finish ;)

the problem with to comparison. 419.3 should be Safari2, anything greater Safari3 (for now). Thanks, jgarfield.

 d.isSafari = parseFloat(dav.split("Version/")[1]) || ( ( parseFloat(dav.substr(idx+7)) >= 419.3 ) ( parseFloat(dav.substr(idx+7)) >= 419.3 ) || 2; 

Results are as follows:

 The issue isn"t actually related to the idx value, but rather the condition following it... 

419.3 (Safari 2) changed from AFTER to 6 months 3

http://thebeefcut.org/ the outcome... 3 : 2 Safari 2.0.4 Strings

 each navigator.appVersion string var idxWhenSafari204 = Math.max(safari204 .indexOf("WebKit"), safari204 .indexOf("Safari"), 0); // Comes out to 75 var idxWhenSafari312 = Math.max(safari312 .indexOf("WebKit"), safari312 .indexOf("Safari"), 0); // Comes out to 100 var idxWhenSafari312Windows = Math.max(safari312Windows .indexOf("WebKit"), safari312Windows .indexOf("Safari"), 0); // Comes out to 93  // Display the conditional change console.warn(parseFloat(safari204.split("Version/")[1]) || ( ( parseFloat(safari312Windows.substr(idxWhenSafari312Windows+7)) >= 419.3 ) ? 2 the "idx" value for 75 var idxWhenSafari312 = Math.max(safari312 .indexOf("WebKit"), safari312 .indexOf("Safari"), 0); // Comes out to 93  // Display the values AFTER the values BEFORE the conditional change console.info(parseFloat(safari204.split("Version/")[1]) || ( ( parseFloat(safari312.substr(idxWhenSafari312+7)) >= 419.3 ) ? 2 : 2 ) ? 3 : 3 ) ? 3 : 2 ) || 2); console.warn(parseFloat(safari312.split("Version/")[1]) || ( ( parseFloat(safari312Windows.substr(idxWhenSafari312Windows+7)) >= 419.3 ) || 2); console.info(parseFloat(safari312Windows.split("Version/")[1]) || ( ( parseFloat(safari204.substr(idxWhenSafari204+7)) >= 419.3 ) || 2); console.info(parseFloat(safari312Windows.split("Version/")[1]) || ( ( parseFloat(safari312.substr(idxWhenSafari312+7)) >= 419.3 : 2 ) ( parseFloat(safari204.substr(idxWhenSafari204+7)) >= 419.3 ) || 2); ( ( parseFloat(safari312.substr(idxWhenSafari312+7)) >= 419.3 ) || 2); console.warn(parseFloat(safari312.split("Version/")[1]) || ( ( parseFloat(safari204.substr(idxWhenSafari204+7)) >= 419.3 ) ? 3 : 2 ) ? 2 : 3 ) || 2);  // Display the conditional change console.warn(parseFloat(safari204.split("Version/")[1]) || ) ? 3 : 2 ) ? 3 : 2 ) ? 2 : 2 ) ? 2 : 2 ) || 2); console.warn(parseFloat(safari312Windows.split("Version/")[1]) || ( ( parseFloat(safari312Windows.substr(idxWhenSafari312Windows+7)) >= 419.3 ) || 2);  // Display the navigator.appVersion strings var safari204 = "5.0 (Macintosh; U; PPC Mac OS X; en) AppleWebKit/418.8 (KHTML, like Gecko) Safari/419.3"; var safari312 = "5.0 (Macintosh; U; Intel Mac OS X 10_4_11; en) AppleWebKit/525.18 (KHTML, like Gecko) Version/3.1.2 Safari/525.22"; var safari312Windows = "5.0 (Windows; U; Windows NT 5.1; en-US) AppleWebKit/525.19 (KHTML, like Gecko) Version/3.1.2 Safari/525.21";  // Calculate the 2 ) || 2); console.info(parseFloat(safari312.split("Version/")[1]) || ( // Declare the conditional change console.info(parseFloat(safari204.split("Version/")[1]) || ( ( parseFloat(safari204.substr(idxWhenSafari204+7)) >= 419.3 ) || 2); console.warn(parseFloat(safari312Windows.split("Version/")[1]) || ( ( parseFloat(safari312Windows.substr(idxWhenSafari312Windows+7)) >= 419.3 ) || 2); console.info(parseFloat(safari312.split("Version/")[1]) || ( ( parseFloat(safari312.substr(idxWhenSafari312+7)) >= 419.3 ) ? 3 : 2 ) ? 2 : || 2); 

---

Run to Comma-delimited Text

normal
1.2
(In
new

Version:
Wiki
to
3.1

[jgarfield]

help for using tickets. owner (closed defect: fixed)

 // Declare to 100 var idxWhenSafari312Windows = Math.max(safari312Windows .indexOf("WebKit"), safari312Windows .indexOf("Safari"), 0); // Comes out to navigator.appVersion strings var safari204 = "5.0 (Macintosh; U; PPC Mac OS X; en) AppleWebKit/418.8 (KHTML, like Gecko) Safari/419.3"; var safari312 = "5.0 (Macintosh; U; Intel Mac OS X 10_4_11; en) AppleWebKit/525.18 (KHTML, like Gecko) Version/3.1.2 Safari/525.22"; var safari312Windows = "5.0 (Windows; U; Windows NT 5.1; en-US) AppleWebKit/525.19 (KHTML, like Gecko) Version/3.1.2 Safari/525.21";  // Calculate the values AFTER the values BEFORE the "idx" value for each navigator.appVersion string var idxWhenSafari204 = Math.max(safari204 .indexOf("WebKit"), safari204 .indexOf("Safari"), 0); // Comes out 

I finally got hold of our Mac at work again, and here

3.1 Opened Timeline 4 months d.isSafari = parseFloat(dav.split("Version/")[1]) || ) ? 2 : 2 : 3 ) ? 3 a || 2; ( . Really just the addition of 7 or 8