#2802 closed bug (fixed)
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 (5)
Changed 12 years ago by
| Attachment: | ticket2802_safari_curCSS.diff added |
|---|
comment:1 Changed 12 years ago by
comment:2 Changed 12 years ago by
| 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
comment:3 Changed 12 years ago by
| 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.
Note: See
TracTickets for help on using
tickets.
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.