thomthom / SKUI

Ruby wrapper of classes that maps to GUI controls in SketchUp's UI::WebDialogs
MIT License
26 stars 14 forks source link

"Newer version of Netscape is required"... :s #92

Closed thomthom closed 10 years ago

thomthom commented 10 years ago

2014-03-14_08h50_13

Strange user report. Is it the user agent string of older IE versions that spoofs it's identity? Or some other software that has changed it?

noelwarr commented 10 years ago

I had no idea SKUI required some minimum IE version. That's great. Any idea what the "is" in Netcapeis is?

On 14 March 2014 08:51, Thomas Thomassen notifications@github.com wrote:

[image: 2014-03-14_08h50_13]https://f.cloud.github.com/assets/192418/2418793/5dfaf4e8-ab4d-11e3-9ae3-ec1a29439fca.png

Strange user report. Is it the user agent string of older IE versions that spoofs it's identity? Or some other software that has changed it?

Reply to this email directly or view it on GitHubhttps://github.com/thomthom/SKUI/issues/92 .

thomthom commented 10 years ago

Missing space typo.

Looks like MS changed the useragent string in IE11 again: http://msdn.microsoft.com/en-us/library/ie/bg182625(v=vs.110).aspx

Also the navigator.appName was changed - which might explain Netscape.

But the user agent string doesn't match what I get in the webdialog:

Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.2; WOW64; Trident/7.0; .NET4.0E; .NET4.0C; .NET CLR 3.5.30729; .NET CLR 2.0.50727; .NET CLR 3.0.30729) SketchUp/14.0 (PC)

Sigh... I'm not getting a reliable version number here.

I expect the change is because there are so many broken IE detection that checked only the first character of the version so it picks up IE10 and IE11 as 1 - but this change breaks the correct detection.... argh!

noelwarr commented 10 years ago

lol.

Doesn't it make you want to take cover in all that SU C++ code? I hate the DOM. Not with a passion because I haven't had to deal with it that much. But I don't think I could live as a web developer without turning to something like this... http://www.ispot.tv/ad/7oAZ/abilify-add-abilify

On 14 March 2014 09:10, Thomas Thomassen notifications@github.com wrote:

Missing space typo.

Looks like MS changed the useragent string in IE11 again: http://msdn.microsoft.com/en-us/library/ie/bg182625(v=vs.110).aspx

Also the navigator.appName was changed - which might explain Netscape.

But the user agent string doesn't match what I get in the webdialog: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.2; WOW64; Trident/7.0; .NET4.0E; .NET4.0C; .NET CLR 3.5.30729; .NET CLR 2.0.50727; .NET CLR 3.0.30729) SketchUp/14.0 (PC)

Sigh... I'm not getting a reliable version number here.

I expect the change is because there are so many broken IE detection that checked only the first character of the version so it picks up IE10 and IE11 as 1 - but this change breaks the correct detection.... argh!

Reply to this email directly or view it on GitHubhttps://github.com/thomthom/SKUI/issues/92#issuecomment-37624654 .