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

Remove Obsoleted Android 4.0 Code #712

Closed
wants to merge 1 commit into from

Conversation

codysherman
Copy link

@codysherman codysherman commented Nov 18, 2017

A super simple, one line change to the specificity of a button rule, removing an old, no longer needed workaround.

This issue is in regards to the current line 278:
html [type="button"], /* 1 */
with the note:
1. Prevent a WebKit bug where (2) destroys native audio and video controls in Android 4.

This pull request returns the line to its original form of just: [type="button"] as it was intended to be. The issue itself has been obsoleted by Google: https://issuetracker.google.com/issues/36950539 More recent Android updates have resolved the bug.

This bug affected Android 4.0 from 2011. This browser version is outside of Normalize's documented Browser Support. Besides being beyond 2 versions of the browser version in the past, it is 6 whole OS versions behind the current version of Android. (And will be 7 next month). This version of the OS is no longer supported by even Google.

Remove workaround for Android Open Source Browser in Android 4.0 fixing media buttons, which is obsoleted by Google and out of Normalize's Browser Support Guidelines.
@garrettw
Copy link

Good catch! I’ll include this change in my derivative library.

@jonathantneal
Copy link
Contributor

I’d love to see this normalization pulled. If it’s outside Normalize.css support, we should merge this and celebrate! 🎉 Thanks, @codysherman .

@necolas necolas closed this in 004d58b Feb 8, 2018
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