Skip to content

fix(hub-ui): preserve toast background on hover - #316

Open
dvcolomban wants to merge 3 commits into
devframes:mainfrom
dvcolomban:codex/fix-toast-hover
Open

fix(hub-ui): preserve toast background on hover#316
dvcolomban wants to merge 3 commits into
devframes:mainfrom
dvcolomban:codex/fix-toast-hover

Conversation

@dvcolomban

@dvcolomban dvcolomban commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • keep the toast background, hover state, rounded corners, and color transition on the outer toast element
  • match the dock's bg-dock-glass material at rest through a bg-toast-glass shortcut
  • strengthen that material on hover without replacing it: 65% to 78% opacity in light mode, and 80% to 90% in dark mode

Before / after

Both screenshots show the upstream messages-toastoverlay--with-action Storybook story in dark mode while hovered.

Before After
hover:bg-active replaces the dark glass surface with a lightly tinted transparent background. The rounded root keeps the dark glass surface and transitions from 80% to 90% opacity.
Toast hover before: glass background is replaced Toast hover after: the root keeps its dark glass background

Theme regression check

These screenshots show the upstream messages-toastoverlay--stack story with the pointer over the first toast. The toast uses the same resting material as the dock in each theme.

Light hover Dark hover
rgba(255, 255, 255, 0.78) rgba(17, 17, 17, 0.9)
Light-theme toast stack with the first toast hovered Dark-theme toast stack with the first toast hovered

Validation

  • pnpm --filter @devframes/hub-ui build
  • pnpm --filter @devframes/hub-ui typecheck
  • pnpm exec eslint packages/hub-ui/uno.config.ts packages/hub-ui/src/client/components/display/ToastOverlay.vue
  • pnpm exec vitest run packages/hub-ui/src (32 passed)
  • Storybook hover checked in light and dark modes
  • computed light background changes from rgba(255, 255, 255, 0.65) to rgba(255, 255, 255, 0.78)
  • computed dark background changes from rgba(17, 17, 17, 0.8) to rgba(17, 17, 17, 0.9)

@vercel

vercel Bot commented Aug 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
devframe Skipped Skipped Aug 28, 2026 1:47pm

@dvcolomban
dvcolomban marked this pull request as ready for review August 28, 2026 11:38
Copilot AI lite review requested due to automatic review settings August 28, 2026 11:38

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adjusts the hub UI toast hover styling so the toast’s “glass” background isn’t replaced on hover, aligning hover feedback with the toast content rather than the container.

Changes:

  • Removes the hover background class from the toast container.
  • Adds hover background styling to the MessageItem content area instead.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread packages/hub-ui/src/client/components/display/ToastOverlay.vue Outdated
@dvcolomban
dvcolomban marked this pull request as draft August 28, 2026 12:15
@dvcolomban
dvcolomban force-pushed the codex/fix-toast-hover branch from 9f5c311 to f579287 Compare August 28, 2026 12:51
@dvcolomban
dvcolomban marked this pull request as ready for review August 28, 2026 13:21
Copilot AI review requested due to automatic review settings August 28, 2026 13:21

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Comment thread packages/hub-ui/uno.config.ts Outdated
@dvcolomban
dvcolomban force-pushed the codex/fix-toast-hover branch from f579287 to 31d5a75 Compare August 28, 2026 13:47
Copilot AI review requested due to automatic review settings August 28, 2026 13:47

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants