Skip to content

Commit

Permalink
[mediaqueries] Remove hover:on-demand
Browse files Browse the repository at this point in the history
  • Loading branch information
frivoal committed Apr 3, 2016
1 parent ffe57f8 commit 2078b46
Showing 1 changed file with 7 additions and 8 deletions.
15 changes: 7 additions & 8 deletions mediaqueries/Overview.bs
Expand Up @@ -1647,7 +1647,7 @@ hover</h3>

<pre class='descdef mq'>
Name: hover
Value: none | on-demand | hover
Value: none | hover
For: @media
Type: discrete
</pre>
Expand All @@ -1666,12 +1666,11 @@ hover</h3>
or there is no pointing system.
Examples include touchscreens and screens that use a drawing stylus.

<dt><dfn>on-demand</dfn>
<dd>
Indicates that the primary pointing system can hover,
but it requires a significant action on the user's part.
For example, some devices can't normally hover,
but will activate hover on a “long press”.
Pointing systems that can hover,
but for which doing so is inconvenient and not part of the normal way they are used,
also match this value.
For example, a touchscreen where a long press is treated as hovering
would match ''hover: none''.

<dt><dfn>hover</dfn>
<dd>
Expand Down Expand Up @@ -1717,7 +1716,7 @@ any-pointer and any-hover</h3>

<pre class='descdef mq'>
Name: any-hover
Value: none | on-demand | hover
Value: none | hover
For: @media
Type: discrete
</pre>
Expand Down

4 comments on commit 2078b46

@RByers
Copy link
Contributor

@RByers RByers commented on 2078b46 Oct 11, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just noticed this had been removed now after some developer confusion on twitter. I filed this bug to explore whether we can remove our usage of this in blink but that'll depend on the web compat impact.

Were bugs filed for any other browsers that support this when it was removed?

@patrickhlauke
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@patrickhlauke
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Firefox does not support this yet, but for completeness added a reference to https://bugzilla.mozilla.org/show_bug.cgi?id=1035774

@frivoal
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know about bugs filed on browser, but if anyone is looking for context, this was discussed here: https://lists.w3.org/Archives/Public/www-style/2016Feb/0041.html and https://lists.w3.org/Archives/Public/www-style/2016Mar/0356.html

Please sign in to comment.