Skip to content

feat: colour the cursor with the accent unless a fill is set - #361

Open
ctate wants to merge 1 commit into
ctate/overlay-label-pillfrom
ctate/overlay-fill-follows-accent
Open

ctate wants to merge 1 commit into
ctate/overlay-label-pillfrom
ctate/overlay-fill-follows-accent

Conversation

@ctate

@ctate ctate commented Oct 31, 2026 •

Copy link
Copy Markdown
Contributor
  • Draw the cursor body in the accent colour by default, so one --accent colours the whole cursor
  • Keep --fill for a body colour other than the accent, and leave it out of the style when unset
  • Update the help, docs and skill that described a white default

Before / after with only --accent set.

The cursor body white before and in the accent colour after

@vercel

vercel Bot commented Oct 31, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
agent-desktop Ready Ready Preview, v0 Oct 31, 2026 8:21am UTC

@ctate
ctate force-pushed the ctate/overlay-fill-follows-accent branch from b1bdf9c to 7e34f93 Compare October 31, 2026 02:08
@ctate
ctate marked this pull request as ready for review October 31, 2026 02:20
@ctate
ctate force-pushed the ctate/overlay-fill-follows-accent branch 2 times, most recently from 5bfd3e4 to 66276e4 Compare October 31, 2026 04:53
@ctate
ctate force-pushed the ctate/overlay-fill-follows-accent branch from 66276e4 to 301e20f Compare October 31, 2026 04:54
@lahfir
lahfir self-requested a review October 31, 2026 05:49
self.fill
.as_deref()
.and_then(rgb)
.unwrap_or_else(|| self.accent_rgb())

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this breaks the Linux live test at cursor_overlay_tests.rs:111, which still expects a white body.

fix: set_fill("#FFFFFF") on the style in that test so it still checks white and doesn't depend on the default.

@ctate
ctate force-pushed the ctate/overlay-fill-follows-accent branch from 301e20f to 79b1a1b Compare October 31, 2026 08:03
- Draw the cursor body in the accent colour by default, so one `--accent` colours the whole cursor
- Keep `--fill` for a body colour other than the accent, and leave it out of the style when unset
- Update the help, docs and skill that described a white default
@ctate
ctate force-pushed the ctate/overlay-fill-follows-accent branch from 79b1a1b to 1b90ed9 Compare October 31, 2026 08:20
@ctate
ctate requested a review from lahfir October 31, 2026 08:35

This branch has not been deployed

No deployments
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