WebDialog.set_html fails under Safari 5.0.6

fredo6

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

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

@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

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

test.zip

Pout

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

driven

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

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

Pout

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

@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

Pout

driven, thx!!
I have reported the error an already received a debugged version (bug at their side)