Side navigation
#11924 closed enhancement (invalid)
Opened June 18, 2012 02:39AM UTC
Closed June 19, 2012 03:26AM UTC
Add cross-browser fix for Opera 11.x regarding onhashchange
Reported by: | Krinkle | Owned by: | Krinkle |
---|---|---|---|
Priority: | low | Milestone: | None |
Component: | event | Version: | 1.7.2 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
Opera 11.1 and 11.5 (maybe more subversions) support onhashchange, but only via the window.onhashchange property, not via window.addEventListener.
I suggest we add a special handling for this event, just like we do for onbeforeunload.
Attachments (0)
Change History (4)
Changed June 18, 2012 02:57AM UTC by comment:1
Changed June 18, 2012 11:25PM UTC by comment:2
component: | unfiled → event |
---|---|
owner: | → Krinkle |
priority: | undecided → low |
status: | new → assigned |
Changed June 19, 2012 03:12AM UTC by comment:3
You may close this as "invalid". The addEventListener is working in Opera 11.x fine. It wasn't working because my sample code omitted the third argument (usually "false") to addEventListener.
Opera doesn't give any feedback that it is required, just silently fails. Either way, as far as jQuery is concerned this is not a bug and the situation is fixed by simply passing false as the third argument (or using jQuery to begin with, in which case that wouldn't happen).
Changed June 19, 2012 03:26AM UTC by comment:4
resolution: | → invalid |
---|---|
status: | assigned → closed |
Cool, thanks for looking into this!
https://github.com/jquery/jquery/pull/832