fix: show hover actions on touch devices#817
Conversation
Signed-off-by: Matt Toohey <contact@matttoohey.com>
Signed-off-by: Matt Toohey <contact@matttoohey.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 385d05eea1
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| opacity: 1; | ||
| } | ||
|
|
||
| @media (hover: none), (pointer: coarse) { |
There was a problem hiding this comment.
Include secondary coarse pointers
This media query only matches when the primary pointer is coarse or the primary input cannot hover. On hybrid devices such as touchscreen laptops/Surfaces with a mouse or trackpad as the primary input, users can still tap the UI but (pointer: coarse) and (hover: none) are false, so these controls remain opacity: 0 and the touch problem persists here and in the repeated media blocks. Use any-pointer: coarse/any-hover: none if the goal is to support any touch-capable input.
Useful? React with 👍 / 👎.
Summary:
Tests: