search

fgelinas / timepicker

A jQuery UI Time Picker
http://fgelinas.com/code/timepicker
GNU General Public License v2.0
266 stars 84 forks source link

I've replaced a deprecated call to .browser #63

Closed tasoili closed 11 years ago

tasoili commented 11 years ago

Since this was a check for Opera, and Opera announced they are moving to webkit, I figured it would be more appropriate to fix based on the engine (presto).

Synchro commented 11 years ago

While this will technically fix the jQuery 1.9 breakage, it's not very elegant. The right way to do it would be to use $.supports, however, I don't know exactly what it is that you'd need to check support for.

tasoili commented 11 years ago

I agree... I figured it was enough to get past the runtime error but does need more work.

I'll try spend a bit more time to work out how the difference may be detectable.