Side navigation
#2802 closed bug (fixed)
Opened May 05, 2008 11:28PM UTC
Closed May 15, 2008 02:46PM UTC
Last modified May 15, 2008 07:30PM UTC
jQuery.curCSS failes in Safari 2.x
Reported by: | ebartels | Owned by: | |
---|---|---|---|
Priority: | undecided | Milestone: | 1.2.4 |
Component: | core | Version: | 1.2.3 |
Keywords: | safari getComputedStyle type error | Cc: | |
Blocked by: | Blocking: |
Description
changeset [5274] introduces an error in Safari 2.x where calls to jQuery.curCSS will fail with a ''type error''
Apparently, Safari 2 doesn't like the use of ''getComputedStyle'' function, introduced on line 615, in place of the full ''document.defaultView.getComputedStyle''
Attachments (1)
Change History (4)
Changed May 05, 2008 11:37PM UTC by comment:1
Changed May 06, 2008 05:57PM UTC by comment:2
need: | Review → Test Case |
---|
I don't see how the optimization is kept.
This needs some testing, but if fails on Safari 2, then we could have a special case where getComputedStyle is a function, that calls internally to document.defaultView.....
Only for Safari 2.x
Changed May 15, 2008 02:46PM UTC by comment:3
resolution: | → fixed |
---|---|
status: | new → closed |
This should be fixed now ([5609]). Could you confirm this ? I don't have Safari 2.
Reopen if it still fails.
Changed May 15, 2008 07:30PM UTC by comment:4
Fixed - that was just in time for me. Thanks Ariel :-)
The patch should still take advantage of the optimization from [5274], but when actually making the function call, it uses the full version to keep Safari happy.