fix(links): paint links in full-screen programs too, in a calmer blue (#97) - #98
Merged
Merged
Conversation
…ullscreen view draws (#97) xterm has no markers, so no decorations, on the alternate screen. Each row drawn there has its link text wrapped in a span of the link colour. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FHHaWKR4M5QtW7Wecyuk4t
The owner found #4ea1ff too saturated. Same hue and lightness, half the saturation. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FHHaWKR4M5QtW7Wecyuk4t
…nate-screen # Conflicts: # core/package.json # package-lock.json # package.json
github-actions Bot
pushed a commit
that referenced
this pull request
Sep 29, 2026
…#97) (#98) * fix(links): paint links on the alternate screen too, where Claude's fullscreen view draws (#97) xterm has no markers, so no decorations, on the alternate screen. Each row drawn there has its link text wrapped in a span of the link colour. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FHHaWKR4M5QtW7Wecyuk4t * style(links): a calmer link blue, #79a7d8 (#97) The owner found #4ea1ff too saturated. Same hue and lightness, half the saturation. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FHHaWKR4M5QtW7Wecyuk4t --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Maxaubert
added a commit
that referenced
this pull request
Sep 29, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #97.
A link in Claude Code's fullscreen view was clickable but never blue. The link painter skipped the alternate screen, which is where full-screen programs draw. xterm has no markers there, so the decorations the normal screen uses cannot exist.
The fix: on the alternate screen, each row xterm draws (
onRender) has its link text wrapped in a span of the link colour, with the same faint underline.A calmer link blue (owner: "make the link colours less blue ... too saturated").
LINK_BLUEgoes from#4ea1ffto#79a7d8: the same hue and lightness at half the saturation. On a light theme it is still moved to the 4.5:1 floor, as before..e2e-shots/links.pngwas checked by eye.Core change, so Prism gets it too through the automatic bump. Core 0.16.1, app 0.19.1. These numbers overlap with the other open PRs, and whichever merges later gets bumped again.
Verified:
linkse2e now enters the alternate screen in a real pwsh, prints a link, and asserts it wears the link blue. This failed on main. It also asserts that a row rewritten in place keeps no link colour.edgesfailed once on its relaunch step, which this change does not touch, and passed on a rerun.🤖 Generated with Claude Code
https://claude.ai/code/session_01FHHaWKR4M5QtW7Wecyuk4t