Skip to content

feat(nav): Increase hover preview delay #93707

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

Merged
merged 1 commit into from
Jun 17, 2025
Merged

Conversation

malwilley
Copy link
Member

In an attempt to address concerns about accidental triggers of the hover preview, this increases the preview delay from 150ms to 300ms.

@malwilley malwilley requested a review from a team June 17, 2025 16:18
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jun 17, 2025
@@ -46,6 +50,7 @@ export function NavContextProvider({children}: {children: React.ReactNode}) {
NAV_SIDEBAR_COLLAPSED_LOCAL_STORAGE_KEY,
false
);
const [collapsedNavIsOpen, setCollapsedNavIsOpen] = useState(false);
Copy link
Member Author

Choose a reason for hiding this comment

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

Because the preview delay is now less than the collapsed nav open delay, I need to put the open state in context so that I can check it in useActivateNavGroupOnHover() to avoid a bad experience where the contents immediately flip on open

@malwilley malwilley merged commit 36f5bba into master Jun 17, 2025
46 checks passed
@malwilley malwilley deleted the malwilley/feat/nav-hover branch June 17, 2025 16:44
andrewshie-sentry pushed a commit that referenced this pull request Jun 19, 2025
In an attempt to address concerns about accidental triggers of the hover
preview, this increases the preview delay from 150ms to 300ms.
@github-actions github-actions bot locked and limited conversation to collaborators Jul 3, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants