WebDialog.set_html fails under Safari 5.0.6

[thomthom]

Fixed a small bug in the sample code.

[Dan Rathbun]

Not personal.. but I'd prefer to rely on "tmpdir.rb" in the Standard Lib.

[thomthom]

@dan rathbun said:

Not personal.. but I'd prefer to rely on "tmpdir.rb" in the Standard Lib.

Sure, it's just a bare bone example. In fact, it'd probably be best to use tempfile.rb to create the temp file as well.

[thomthom]

I did some further testing I found some issues which I addressed in my example: http://forums.sketchucation.com/viewtopic.php?f=180&t=39842&p=352375#p352375

.set_file doesn't cause the content of the webdialog to update if the filename is the same. So I made the example generate a new file with a unique seed in the filename to ensure it's updated.

.set_file and .set_url now immediately cleans up the temp file if .set_html was used prior.

No more references to my TT_Lib in the example.

Linked to the Standard Lib in the comments for more robust temp file handlers.

[Dan Rathbun]

Can these meta tags be injected to prevent caching:
%(#8000BF)[<META HTTP-EQUIV="Pragma" CONTENT="no-cache"> <META HTTP-EQUIV="Expires" CONTENT="-1">]

ref: How to prevent caching in Internet Explorer

[thomthom]

@dan rathbun said:

Can these meta tags be injected to prevent caching:
%(#8000BF)[<META HTTP-EQUIV="Pragma" CONTENT="no-cache"> <META HTTP-EQUIV="Expires" CONTENT="-1">]

ref: How to prevent caching in Internet Explorer

I did consider sending a refresh comment to JS, but I figured I'd follow the pattern of how .set_html works on Windows, which is creating a temp file with a unique name.

[Dan Rathbun]

@dan rathbun said:

Has anyone logged this with Google ?? I'd think they'd wish to release patches if Safari 5.0.6 will break WebDialogs on all Mac versions.

TT ?

[thomthom]

I've not logged anything yet. Been bugging a couple to see if they could shed some light.

[thomthom]

Driven: I implemented a patch. Update to TT_Lib 2.5.5
http://forums.sketchucation.com/viewtopic.php?f=323&t=30503

[fredo6]

To all,

Can we summarize what is the problem and what could be the workaround.

From what Driven has signaled, there are problems in both 5.0.3 and 5.0.6 Safari versions.

Also it would be good to hear from the Google SU Team, as they can perfectly implement a central solution that makes set_htmlwork.

Fredo

[Dan Rathbun]

@thomthom said:

I've not logged anything yet. Been bugging a couple to see if they could shed some light.

OK.. then your on "top of this"... I suppose "they'll" ask you to log a formal bug # if they don't do it themselves.

@Fredo.. agreed, and I hope they'll monitor this topic for ideas.

[thomthom]

@unknownuser said:

Can we summarize what is the problem and what could be the workaround.
From what Driven has signaled, there are problems in both 5.0.3 and 5.0.6 Safari versions.

I tried to summarize here: http://forums.sketchucation.com/viewtopic.php?f=180&t=39842&p=352375#p352375

Not sure about 5.0.3 - but there has been some discussions on how to format the URLs linking to local resources. We found that file:/// worked on both platforms.

[thomthom]

@dan rathbun said:

@thomthom said:

I've not logged anything yet. Been bugging a couple to see if they could shed some light.

OK.. then your on "top of this"... I suppose "they'll" ask you to log a formal bug # if they don't do it themselves.

Submitted!

[fredo6]

@thomthom said:

Not sure about 5.0.3 - but there has been some discussions on how to format the URLs linking to local resources. We found that file:/// worked on both platforms.

The issue with file:// on Mac/Safari was already there since the beginning.

So at the end, the workaround is just to use set_file on Mac

• which works on Safari 5.06 (and to be confirmed on 5.03)
• which also works on previous versions, though set_html still works in Safari up to 5.01

Is this correct?

Fredo

[thomthom]

I've used a mix of .set_html and .set_file in my plugins and have not had any problems until 5.0.6 came out.

[thomthom]

@driven said:

TT_SelectionToys works EXCEPT for 'cheat sheet' which shows as about:blank and loads nothing, not even error messages, so may be a different issue.

I believe it's a different issue as it's an older plugin. Lets look away from this one.

@driven said:

TT_CleanUp3 all now work with the latest patch EXCEPT temp file are not purged on close of dialog or SU.

Really? They should be erased when SketchUp is exited. hmm... Gonna look at it again on my Mac.

[driven]

on my computers, both running OSX 10.5.6.

under Safari 5.0.3

Fredo's WD all work correctly.
TT_SelectionToys works EXCEPT for 'cheat sheet' which shows as about:blank and loads nothing, not even error messages, so may be a different issue.
TT_CleanUp3 all works with original file (will test new one now) UPDATE: new file works

under Safari 5.0.6
Fredo's WD failed to load images and failed on check for updates . with the temp file test, it will now switch from about:blank if a manual reset is induced, It THEN works correctly.
TT_SelectionToys works EXCEPT [as above]
TT_CleanUp3 all now work with the latest patch EXCEPT temp file are not purged on close of dialog or SU.

from my testing .set_file and .set_url seem to be interchangeable (with the corrected path), i.e. the same resource from the same source can be used with either method.

john

[driven]

@TT new patch works on Safari 5.0.3 as well, updated above... and temp files also don't purge.

[thomthom]

It appears that the Garbage Collector is no invoked when SketchUp exits under OSX - as oppose to under Windows. Gonna have to look for another way to cleanup.

[thomthom]

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.