Thanks to visit codestin.com
Credit goes to github.com

Skip to content

in-house tooltips & floating menus - #7495

Merged
mergify[bot] merged 2 commits into
mainfrom
prospector/new-tooltips
Sep 13, 2026
Merged

in-house tooltips & floating menus#7495
mergify[bot] merged 2 commits into
mainfrom
prospector/new-tooltips

Conversation

@Prospector

@Prospector Prospector commented Sep 12, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Closes #1679
Closes #6580

@github-actions

github-actions Bot commented Sep 12, 2026 •
edited by Prospector
Loading

Copy link
Copy Markdown
Contributor

Pull request changelog

App

Added

Changed

  • Tooltips now appear only after a delay, no longer re-open when quickly moving to another nearby tooltip, smoothly animate between positions, and have a slight delay before closing to bring them closer to typical tooltip guidelines. Tooltip animations have also been improved to be smoother.

Deprecated

Removed

Fixed

  • Fixed focus trapping of modal menus.
  • Fixed the action bar in the modpack content modal displaying below the modal.

Security

Website

Added

Changed

  • Tooltips now appear only after a delay, no longer re-open when quickly moving to another nearby tooltip, smoothly animate between positions, and have a slight delay before closing to bring them closer to typical tooltip guidelines. Tooltip animations have also been improved to be smoother.
  • Added a slight selection animation to selecting the different tabs of the app.

Deprecated

Removed

Fixed

  • Fixed focus trapping of modal menus.
  • Fixed many screen reader issues. A lot more work still needs to be done here, but certain parts of the app should be much better with screen readers now.

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

Security

@github-actions

github-actions Bot commented Sep 12, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Frontend previews

Last deployed commit is f36bc3e8807694978c8c2438ea67a2e09a0de4d5

Persistent

Environment URL
staging https://pr-7495-staging-frontend.modrinth.workers.dev
production https://pr-7495-frontend.modrinth.workers.dev
storybook https://pr-7495-storybook.modrinth.workers.dev

Latest commit

Environment URL
staging https://git-f36bc3e8-staging-frontend.modrinth.workers.dev
production https://git-f36bc3e8-frontend.modrinth.workers.dev
storybook https://git-f36bc3e8-storybook.modrinth.workers.dev

@Prospector
Prospector requested review from IMB11 and tdgao and removed request for IMB11 September 12, 2026 21:44
@IMB11

IMB11 commented Sep 13, 2026

Copy link
Copy Markdown
Member

@mergify merge

@mergify

mergify Bot commented Sep 13, 2026

Copy link
Copy Markdown

merge

❌ Sorry but I didn't understand the command. Please consult the commands documentation 📚.

@mergify

mergify Bot commented Sep 13, 2026 •
edited
Loading

Copy link
Copy Markdown

Queued — the merge queue status continues in this comment ↓.

@mergify

mergify Bot commented Sep 13, 2026 •
edited
Loading

Copy link
Copy Markdown

Merge Queue Status

  • ✅ Entered queue — 2026-09-13 22:19 UTC · Rule: default · triggered by @Prospector with the merge queue checkbox
  • ✅ Checks skipped · PR is already up-to-date
  • ✅ Merged — 2026-09-13 22:20 UTC · at 18adbcae7c83cafd04a50611cc0f3be3277e9ecd

This pull request spent 11 seconds in the queue, including 1 second running CI.

Required conditions to merge

@mergify mergify Bot added the queued label Sep 13, 2026
@mergify
mergify Bot merged commit 18adbca into main Sep 13, 2026
16 checks passed
@mergify
mergify Bot deleted the prospector/new-tooltips branch September 13, 2026 22:20
@mergify mergify Bot removed the queued label Sep 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@IMB11 IMB11 IMB11 approved these changes
@tdgao tdgao Awaiting requested review from tdgao

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[UX] No tooltip delay for subsequent tooltips Elements behind modals can still be focused

2 participants

@Prospector @IMB11