search

TurnWheel / jReject

jQuery Browser Rejection Plugin
http://jreject.turnwheel.com/
277 stars 98 forks source link

Defective mobile recognition (Android) #50

Open m1n0 opened 10 years ago

m1n0 commented 10 years ago

I noticed that some android browsers contain string "Version/X", e.g. "Mozilla/5.0 (Linux; Android 4.4.3; Galaxy Nexus Build/KTU84M) AppleWebKit/537.36 (KHTML, like Gecko) Version/4.0 Chrome/33.0.0.0 Mobile Safari/537.36"

This string gets mistaken as a chrome version and is rejected. (as Chrome 4 in my example).

The parsing mechanism should be aware of that and deal with it somehow (perhaps ignore it? I dont know what is that part used for.)

cm325 commented 9 years ago

:+1:

bdelaney commented 9 years ago

:+1:

scotself commented 9 years ago

Is there a solution available to this? We're using the Drupal module that uses this library and have to disable all rejection of old Chrome browsers in order for Androids to work. Not 100% sure that solution covers all Androids either as some don'e have the Chrome part of the string (though those that don't seem to be accepted and don't get the popup).

djachenko commented 9 years ago

I've changed underlaying library which detect browser to more up-to-date one within this pull request. It detects mobile browsers, so it can be solution for your problem. Try this version (I don't suppose jReject's authors will merge them in fast, judging by their recent activity) and say if it worked for you, please.