Skip to content

Keep selected editor text visible - #49

Merged
jserv merged 1 commit into
sysprog21:mainfrom
moon-jam:fix-editor-selection-text-visable
Sep 16, 2026
Merged

jserv merged 1 commit into
sysprog21:mainfrom
moon-jam:fix-editor-selection-text-visable

Conversation

@moon-jam

@moon-jam moon-jam commented Sep 16, 2026

Copy link
Copy Markdown
Collaborator

Summary

Selected text in the interview editor was unable to read because the textarea keeps its text transparent while the selection background covers the syntax-highlight layer underneath.

Set a foreground color for selected text so it remains visible while preserving the existing overlay-based syntax highlighting.

Sample

Before:
image

After:
image


Summary by cubic

Fixes selected text in the interview editor being unreadable by adding a foreground color to #editor::selection. The textarea glyphs stay transparent for the syntax-highlight overlay, but the selection background was covering that layer; now selected text uses the ink color and remains visible.

Written for commit efbb220. Summary will update on new commits.

Review in cubic

cubic-dev-ai[bot]

This comment was marked as resolved.

@ColtenOuO ColtenOuO left a comment

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.

Just a minor suggestion, non-blocking for this PR.

Comment thread web/styles.css Outdated
The textarea leaves its glyphs transparent so the syntax-highlight layer
can paint them. Its selection background covers that layer, which made
selected code unreadable without a separate foreground color.
@moon-jam
moon-jam force-pushed the fix-editor-selection-text-visable branch from d9221e4 to efbb220 Compare September 16, 2026 14:03
@jserv
jserv merged commit a8b3c10 into sysprog21:main Sep 16, 2026
6 checks passed
@jserv

jserv commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Thank @moon-jam for contributing!

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.

3 participants