Remove Opera sniff

[miketaylr]

SUP BEN.

Opera 9.64 is like, 9 versions old. I know Ryan has a fork that normalizes its old document.postMessage implementation--but I think that's going to help like 0.01% of all Opera users. So....this is easier and won't break any sites or provide a crappy fallback for current Opera users.

[miketaylr]

Ben, want me to um write a test for this or something?

[miketaylr]

Beatport.com users can't complete the checkout process because of this sniff now too. 😞

[alexmchardy]

We're patched now at Beatport. Thanks for the fix!

[jtsoi]

I was about to make the same pull request, Opera now supports postMesage.

Ben, are you alive? =)

/JT

[chrisdavidmills]

+1 on this from me too!

[alloy-d]

Yet another +1.

[tagawa]

Another +1 here.

[andreasbovens]

And another +1 from me.

[freddywang]

for Opera, use

window.addEventListener('message', callback, false)

instead of

window.addEventListener('message', callback, !1)

I checked on opera 12, it doesn't work when we pass !1 as third argument of addEventListener method

[abhishekdev]

If not removed this "sniffing" statement should be modified for compatibility with jQuery 1.9.x+ as $.browser is deprecated now.

[Calvein]

👍

[lukeasrodgers]

👍

[damienwhaley]

Further to this I have modified the browser sniff so that it does not fail on jQuery 1.9.x+ and I have added a note to the README to use the jQuery.browser plugin.

[miketaylr]

hi ben, miss u. ❤️

[vladikoff]

Yesss!

[buritica]

🎂 happy bday PR