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

[Bug]: Sticky hover effect on mobile #30388

Closed
2 tasks done
daniharo opened this issue Jan 25, 2024 · 1 comment
Closed
2 tasks done

[Bug]: Sticky hover effect on mobile #30388

daniharo opened this issue Jan 25, 2024 · 1 comment

Comments

@daniharo
Copy link
Contributor

daniharo commented Jan 25, 2024

Library

React Components / v9 (@fluentui/react-components)

System Info

System:
    OS: macOS 14.2.1
    CPU: (8) arm64 Apple M1
    Memory: 63.02 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Browsers:
    Safari: 17.2.1
    
NPM packages:
"@fluentui/react-components": "^9.45.0"

Are you reporting Accessibility issue?

no

Reproduction

https://codesandbox.io/p/sandbox/fluent-sticky-hover-cxcx89

Bug Description

Actual Behavior

In Safari mobile or Chromium mobile, after touching a button, it remains being shown in the "hover" styles.

image

[Edit @behowell] This also repros on Edge on Windows when using touch.

Expected Behavior

In mobile browsers, :hover styles should be disabled for all components. There's no obvious way to do this for Fluent v9 components right now.

Logs

No response

Requested priority

Normal

Products/sites affected

No response

Are you willing to submit a PR to fix?

no

Validations

  • Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
  • The provided reproduction is a minimal reproducible example of the bug.
@mainframev
Copy link
Contributor

mainframev commented Jun 28, 2024

Hi @daniharo 👋🏻

Unfortunately, after investigating it and a short discussions we had, the problem reported can't be fixed right now. It requires proper testing and a lot of changes on our side. I created a feature request, so we won't lost it

Thank you for reporting the issue 🫶

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants