Skip to content

style: fullscreen icon, collapse icon, focus ring#7571

Merged
akshayka merged 1 commit into
mainfrom
aka/full-screen-button-nit
Dec 26, 2025
Merged

style: fullscreen icon, collapse icon, focus ring#7571
akshayka merged 1 commit into
mainfrom
aka/full-screen-button-nit

Conversation

@akshayka
Copy link
Copy Markdown
Contributor

@akshayka akshayka commented Dec 19, 2025

  • Fullscreen icon and collapse icon should use the same opacity as the other editor buttons
  • Add a nonzero ring offset to the cell focus ring so it can be seen even on cells with errors; make ring slate to be less jarring, with ring offset keeping it visible

With this change (notice the ring):
image

Without this change (no ring):
image

* Fullscreen icon and collapse icon should use the same opacity as the
  other editor buttons
* Add a ring to the cell focus outline so it can be seen even on cells
  with errors
@vercel
Copy link
Copy Markdown

vercel Bot commented Dec 19, 2025

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

Project Deployment Review Updated (UTC)
marimo-docs Ready Ready Preview, Comment Dec 19, 2025 10:50pm

@akshayka akshayka merged commit 9134a8f into main Dec 26, 2025
26 checks passed
@akshayka akshayka deleted the aka/full-screen-button-nit branch December 26, 2025 17:42
@Light2Dark Light2Dark added the enhancement New feature or request label Jan 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants