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

Deprecate hover media query shim and mixins #25270

Merged
merged 4 commits into from Jan 11, 2018
Merged

Conversation

mdo
Copy link
Member

@mdo mdo commented Jan 11, 2018

Fixes #25195.

This never was completed or fully implemented, and the docs are inaccurate as to what browsers are currently affected by this stickied hover problem. This PR aims to update the docs and deprecate the shim from our mixins and only return the pseudo-classes. To reflect the order of these states used elsewhere (e.g., our button styles), I've also updated the order of each within the mixin.

Fixes #25195.

This never was completed or fully implemented, and the docs are inaccurate as to what browsers are currently affected by this stickied hover problem. This PR aims to update the docs and deprecate the shim from our mixins and only return the pseudo-classes. To reflect the order of these states used elsewhere (e.g., our button styles), I've also updated the order of each within the mixin.
@mdo mdo added this to Inbox in v4.0 stable via automation Jan 11, 2018
@mdo mdo moved this from Inbox to Needs review in v4.0 stable Jan 11, 2018
@patrickhlauke patrickhlauke merged commit 92cc0ab into v4-dev Jan 11, 2018
v4.0 stable automation moved this from Needs review to Shipped Jan 11, 2018
@mdo mdo mentioned this pull request Jan 11, 2018
@mdo mdo deleted the deprecate-hover-mixin branch May 1, 2018 03:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
v4.0 stable
  
Shipped
Development

Successfully merging this pull request may close these issues.

None yet

3 participants