Support for Opera Mini #146

Closed
wants to merge 2 commits into
from

Projects

None yet

2 participants

@fflewddur

Opera Mini and some other mobile browsers use servers to process JavaScript, then send the result back to the device. Devices like these don't support the Scroll event, so I added a check to the constructor that verifies the browser supports 'onscroll'. If it doesn't, we won't hide the manual pagination link. This allows the plugin to fail gracefully on low-power mobile devices.

I've only been able to test this with Opera Mini for iOS, so verification that it works on other devices/browsers would be appreciated.

@fflewddur fflewddur If the user's browser doesn't support the Scroll event, don't hide th…
…e manual pagination link.
03866cc
@fieg
Member
fieg commented Jun 26, 2014

Thank you for your PR. I think the detection code could should be placed in the initialize method around here. I will implement it some other time when I have some more time on my hands.

@fflewddur

Ah, good point. If you don't beat me to it, I'll adjust my branch later this week and update the PR.

@fflewddur fflewddur Move detection code to the initialization method.
1937cc3
@fieg fieg closed this in 49c5774 Jan 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment