Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
# Changelog

## 1.0.90

- Fix: the **`match …` markers are outlined rather than filled.** They carried the same solid amber as the highlighted words, which was right while a row had at most one of them; [#152](https://github.com/grimmerk/codev/pull/152) made a row carry up to four (`match #N` plus `match path`, `match assistant`, and `match recap` or `match reply`), and at that count the labels competed with the text they point at and could not be told apart from it. They are now amber-outlined pills, the shape every other badge on the row already uses (PR, account, terminal), so solid amber means exactly one thing: text the query matched. The `id 4ed7505a` marker follows, keeping the corner radius of the badges beside it on that line. No row is taller for it: the border is drawn on an inline element, which costs no line height

## 1.0.89

- Feat: a search result explains itself and can be walked ([#141](https://github.com/grimmerk/codev/issues/141), [#146](https://github.com/grimmerk/codev/issues/146))
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "CodeV",
"productName": "CodeV",
"version": "1.0.89",
"version": "1.0.90",
"description": "Quick switcher for VS Code, Cursor, and Claude Code sessions",
"repository": {
"type": "git",
Expand Down
41 changes: 30 additions & 11 deletions src/switcher-ui.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -61,16 +61,31 @@ const SEARCH_HIGHLIGHT_STYLE = {
fontWeight: 600,
} as const;

// Deliberately the SAME amber as SEARCH_HIGHLIGHT_STYLE: the chip and the
// highlighted words are one system, so the row reads as "search found this
// here" at a glance. The line's text stays the neutral prompt grey — an amber
// line body sat too close to the orange last-message line, and the snippet IS
// a user prompt, so colouring it as one is also the honest choice.
// The label naming what a snippet line is (`match #N`, `match path`, …).
//
// It used to be FILLED with the same amber as SEARCH_HIGHLIGHT_STYLE, which
// was right while a row carried at most one of them (#139): one chip and the
// highlighted words read as a single "search found this here" system. PR #152
// made a row carry up to four — `match #N` plus path, assistant, and recap or
// reply — and at that count the filled amber competes with the thing it points
// at, since the label and the matched words are then the same solid block of
// colour. Reported as noise, and as "I cannot tell a field label from matched
// text".
//
// So the hue stays (the label is still part of the search system) and the
// weight goes: an outlined pill, which is what every other badge on the row
// already is (PR, account, terminal). Filled amber now means exactly one
// thing: text the query actually matched.
//
// `display` is deliberately left at inline. An inline border costs no line
// height and is not clipped by the line's `overflow: hidden`; inline-block
// adds 2–8px per row, measured 2026-09-08 across four rows of chips.
const SNIPPET_MARKER_STYLE = {
color: SEARCH_HIGHLIGHT_STYLE.color,
backgroundColor: SEARCH_HIGHLIGHT_STYLE.backgroundColor,
borderRadius: '2px',
padding: '0 4px',
// The highlight amber, dimmed for text and at half strength for the border.
color: '#e0b060',
border: '1px solid rgba(245, 185, 66, 0.5)',
borderRadius: '9px',
padding: '0 6px',
fontSize: '10px',
fontWeight: 600,
} as const;
Expand Down Expand Up @@ -3507,8 +3522,12 @@ function SwitcherApp() {
<span
style={{
fontSize: '10px',
color: '#1a1a1a',
backgroundColor: SEARCH_HIGHLIGHT_STYLE.backgroundColor,
// Amber like the other match markers, but the
// 3px corner of the badges it sits among on
// this line (PR, account, terminal) rather
// than the snippet lines' pill.
color: SNIPPET_MARKER_STYLE.color,
border: SNIPPET_MARKER_STYLE.border,
borderRadius: '3px',
padding: '1px 5px',
fontFamily: 'Menlo, monospace',
Expand Down
Loading