Skip to content

fix: suppress dark-mode hover border on published cells#8297

Merged
Light2Dark merged 1 commit intomarimo-team:mainfrom
jeffs:fix-dark-hover
Feb 13, 2026
Merged

fix: suppress dark-mode hover border on published cells#8297
Light2Dark merged 1 commit intomarimo-team:mainfrom
jeffs:fix-dark-hover

Conversation

@jeffs
Copy link
Contributor

@jeffs jeffs commented Feb 13, 2026

Summary

In dark-mode app view, hovering over a published cell adds a 1px border that causes layout jiggle. The existing .dark .marimo-cell:hover rule (specificity 0,3,0) overrides .marimo-cell.published:hover (same specificity, earlier source order). Light mode is unaffected because there's no later light-mode hover rule.

Fix: add &:hover { border: none; box-shadow: none; } inside the existing .dark .marimo-cell &.published block.

Test plan

  • Open any notebook in app view (read mode) with dark theme
  • Hover over cells
  • Verify no border appears and no layout shift occurs

@vercel
Copy link

vercel bot commented Feb 13, 2026

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

Project Deployment Actions Updated (UTC)
marimo-docs Ready Ready Preview, Comment Feb 13, 2026 11:12am

Request Review

@github-actions
Copy link

github-actions bot commented Feb 13, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@jeffs
Copy link
Contributor Author

jeffs commented Feb 13, 2026

I have read the CLA Document and I hereby sign the CLA

@jeffs
Copy link
Contributor Author

jeffs commented Feb 13, 2026

CI requires a label to pass, but I don't have permission to apply one.

@mscolnick mscolnick added the bug Something isn't working label Feb 13, 2026
@Light2Dark Light2Dark merged commit b58b678 into marimo-team:main Feb 13, 2026
26 of 29 checks passed
@jeffs jeffs deleted the fix-dark-hover branch February 13, 2026 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants