WebDialog.set_html fails under Safari 5.0.6

thomthom · 5 Sept 2011, 08:09

Yea, I was planning on adding a large data pump to my library anyway. But I need to work out what the data limit is... Hopefully I can just send a very large string from ruby and compare it in JS.

thomthom · 5 Sept 2011, 08:17

Actually, what I really hope for is that at_exit works under OSX.

Dan Rathbun · 8 Sept 2011, 09:41

@thomthom said:

Cursed - this is more troublesome than I thought.
Under Windows I can clean up all temps by using AppObserver.onQuit - but under OSX it appear to never trigger.

So then there's no way under OSX that my proposed workaround for the Instance Observer can work for SU8.

Makes me wonder if "they" just turned off the onQuit() callback (for OSX,) as a 'quick-temporary-fix' for the BugSplat! on close issue?

thomthom · 8 Sept 2011, 09:58

@dan rathbun said:

Makes me wonder if "they" just turned off the onQuit() callback (for OSX,) as a 'quick-temporary-fix' for the BugSplat! on close issue?

hm... when was that fix issued? I'd like to try with an older SU to check.

Dan Rathbun · 8 Sept 2011, 10:25

@thomthom said:

@dan rathbun said:

Makes me wonder if "they" just turned off the onQuit() callback (for OSX,) as a 'quick-temporary-fix' for the BugSplat! on close issue?

hm... when was that fix issued? I'd like to try with an older SU to check.

I am speculating... remember during v8 beta the Instance Observer OSX BugSplat! on close without saving the model first?
It was not fixed for 8.0M1.

However you have that onQuit() works for v7.1 on OSX.

thomthom · 8 Sept 2011, 10:53

I tried onQuit and at_exit on OSX SU 7.1.6859 and I could not get them to trigger either...

Dan Rathbun · 8 Sept 2011, 11:57

It's a bit confusing to have your Observer table combined with both platforms, have you thot about having a table for each platform?

Dan Rathbun · 8 Sept 2011, 12:00

And just to be clear, your testing using the Ruby ver that ships with the Sketchup version? (v1.8.5-p0)

What happens if your change the Sketchup Ruby symbolic links on the Mac to a newer Ruby install (v1.8.6-p287 or higher?)

Dan Rathbun · 8 Sept 2011, 12:07

And also you might try to trap the SystemExit exeception, which is supposed to be called, unless Google used exit!() in which case your out of luck.

I'm not sure if Kernel.set_trace_func will catch a call to exit!() or not.

thomthom · 8 Sept 2011, 12:33

@dan rathbun said:

It's a bit confusing to have your Observer table combined with both platforms, have you thot about having a table for each platform?

Wouldn't having two tables make it harder to get an overview of what observers has issues?

@dan rathbun said:

And just to be clear, your testing using the Ruby ver that ships with the Sketchup version? (v1.8.5-p0)

What happens if your change the Sketchup Ruby symbolic links on the Mac to a newer Ruby install (v1.8.6-p287 or higher?)

Yes - I only test SketchUp out of the box. And that is complicated and time-consuming enough...

Dan Rathbun · 8 Sept 2011, 13:53

I wish someone could make a one-click dmg installer for Sketchup Mac that would bring their Ruby up to par with the PC edition.

fredo6 · 9 Sept 2011, 17:47

The problem seems to be in all versions after 5.0.6

A user encountered the problem on Safari Version 5.1 (7534.48.3): images not shozing and Update (via xmlhttprequest not working)

Fred

Pout · 20 Sept 2011, 10:40

I'm a bit confused here.

So if you would use set_file then all should work fine?
I'm using that method and it loads a html file but the linked js files inside do not seem to work.
They do however work when just opening the html file with Safari. (So not in a webdialog)

thomthom · 20 Sept 2011, 10:44

@pout said:

I'm a bit confused here.

So if you would use set_file then all should work fine?
I'm using that method and it loads a html file but the linked js files inside do not seem to work.
They do however work when just opening the html file with Safari. (So not in a webdialog)

Yes, set_file should work. Got a sample of what is not working?

Pout · 20 Sept 2011, 10:49

here you go
unzip, put the rb in plugisn and the Grid directory to
thx for testing!

test.zip

Pout · 20 Sept 2011, 10:50

and in SU you can launch by TESTOLA.testing in ruby console

driven · 20 Sept 2011, 13:27

With your WebDialog in Safari 5.0.6 Web Inspector shows this error

TypeError: 'null' is not an object (evaluating 'navigator.userAgent.match(/(?:KHTML\D*|VERSION\D*)(\d[\d\.]+)/i)[1]')

I'd look into that to start.

john

thomthom · 20 Sept 2011, 13:43

hmm... why are you sniffing the userAgent string? It's preferred to test for features instead of testing the user agent string.

Pout · 20 Sept 2011, 14:50

Well it is in fact not a script written by me.
It is an external script which i'm trying to incorporate.
So the complete js side is unclear to me.
In what file is the error located?

Thx again

driven · 20 Sept 2011, 15:52

@pout said:

In what file is the error located?
Thx again

GridE.js
It's in encodedstring LZD="....."

which I can't look at.

john