Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed a regression on webkit family browsers with devicePixelRatio 1 #1554

Merged

Conversation

teebot
Copy link
Contributor

@teebot teebot commented Mar 12, 2015

Shifting the image detection by a few pixels seems to solve a false negative on chrome and safari with devicePixelRatio 1.

Shifting the image detection by a few pixels seems to solve a false
negative on chrome and safari with devicePixelRatio 1.
@ryanseddon
Copy link
Member

An this still returns results correctly in all others?

@teebot
Copy link
Contributor Author

teebot commented Mar 13, 2015

I just retested deviceRatio 1 and 2, it still fails for one browser:
Windows 8
IE 9 to 11 pass
Firefox 36 pass
Chrome 41 fail

OSX
Safari pass
Firefox 36 pass
Chrome 41 pass

Is there a more reliable way to detect emojis than the print/sample method?
The implementation of emojis is so different from an OS/Browser combination to the other that it's maybe worth splitting the heuristics by case. What do you think?

@patrickkettner
Copy link
Member

what do you mean when you say fail? chrome for windows does not support
emoji, so a "fail" would mean you get a positive result there.

On Fri, Mar 13, 2015 at 3:25 AM, Thibaut Nguyen notifications@github.com
wrote:

I just retested deviceRatio 1 and 2, it still fails for one browser:
Windows 8
IE 9 to 11 pass
Firefox 36 pass
Chrome 41 fail

OSX
Safari pass
Firefox 36 pass
Chrome 41 pass

Is there a more reliable way to detect emojis than the print/sample method?
The implementation of emojis is so different from an OS/Browser
combination to the other that it's maybe worth splitting the heuristics by
case. What do you think?


Reply to this email directly or view it on GitHub
#1554 (comment).

patrick

@teebot
Copy link
Contributor Author

teebot commented Mar 13, 2015

oh ok I thought they had implemented it

patrickkettner added a commit that referenced this pull request Mar 13, 2015
Fixed a regression on webkit family browsers with devicePixelRatio 1
@patrickkettner patrickkettner merged commit c653d62 into Modernizr:master Mar 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants