Skip to content

fix(remote-control-status): use a Nerd Fonts v3 codepoint for the off… - #605

Merged
sirmalloc merged 1 commit into
sirmalloc:mainfrom
D-L-Narayana:fix-remote-control-off-glyph
Oct 29, 2026
Merged

sirmalloc merged 1 commit into
sirmalloc:mainfrom
D-L-Narayana:fix-remote-control-off-glyph

Conversation

@D-L-Narayana

Copy link
Copy Markdown
Contributor

… icon

The Nerd Font "off" glyph was U+F6AC, a Nerd Fonts v2 codepoint from the Material Design range that v3 moved to the supplementary plane. Every v3 font renders it as a missing-glyph box, so remote-control-status showed tofu whenever remote control was off while the "on" icon (nf-fa-wifi, U+F1EB) rendered fine.

Use nf-md-wifi_off (U+F05AA), the v3 location of the same icon and the natural pair for nf-fa-wifi. Checked against glyphnames.json from Nerd Fonts 3.5.1: U+F05AA is md-wifi_off and U+F6AC has no glyph.

Fixes #597

… icon

The Nerd Font "off" glyph was U+F6AC, a Nerd Fonts v2 codepoint from the
Material Design range that v3 moved to the supplementary plane. Every v3
font renders it as a missing-glyph box, so `remote-control-status` showed
tofu whenever remote control was off while the "on" icon (nf-fa-wifi,
U+F1EB) rendered fine.

Use nf-md-wifi_off (U+F05AA), the v3 location of the same icon and the
natural pair for nf-fa-wifi. Checked against glyphnames.json from Nerd
Fonts 3.5.1: U+F05AA is md-wifi_off and U+F6AC has no glyph.

Fixes sirmalloc#597
@sirmalloc
sirmalloc merged commit 5f0046a into sirmalloc:main Oct 29, 2026
pcvelz added a commit to pcvelz/ccstatusline-usage that referenced this pull request Nov 2, 2026
Upstream: weekly limit timers (sirmalloc#768), prompt cache miss widgets (sirmalloc#770), context window size env override (sirmalloc#352, sirmalloc#407), windowed speed fixes (sirmalloc#769, sirmalloc#771), separator/renderer fixes (sirmalloc#773), custom-command styling (sirmalloc#775), clickable cwd links (sirmalloc#744), account email from live credentials (sirmalloc#724), cache timer reported expiry + short toggle (sirmalloc#576, sirmalloc#592), claude-status lock (sirmalloc#635, sirmalloc#772), dedup token metrics (sirmalloc#622, sirmalloc#725, sirmalloc#735).
Not adopted: sirmalloc#765 terminal width from COLUMNS with fixed reserve - the fork keeps flexMode/compactThreshold and tmux pane priority.

Hand edits outside conflicts:
- README.md: kept fork Configure flex separator behavior bullet and older-updates details wrapper; dropped upstream Related Projects additions (claude-eta)
- AGENTS.md: kept fork terminal width wording (sirmalloc#765 not adopted)
- src/types/FlexMode.ts, src/tui/components/TerminalWidthMenu.tsx, src/tui/components/__tests__/TerminalWidthMenu.test.ts, src/tui/components/TerminalOptionsMenu.tsx, src/tui/components/__tests__/TerminalOptionsMenu.test.ts: restored, fork keeps terminal width settings
- src/tui/App.tsx, src/tui/components/index.ts, src/types/index.ts: re-wired terminal width menu and FlexMode export
- src/types/StatusJSON.ts: dropped fork duplicate prompt_cache key, upstream schema is a superset
- src/utils/terminal.ts: COLUMNS source gated on non-tmux so tmux pane width keeps priority
- src/widgets/WeeklyLimitTimer.ts, src/utils/usage-prefetch.ts, src/widgets/__tests__/WeeklyLimitTimer.test.ts: compat fix - fableUsage/fableResetAt -> weeklyFableUsage/weeklyFableResetAt
- src/widgets/ApiUsage.tsx: compat fix - Context Bar honors effective context window (override, clamp, 1.0M format); Weekly renders a valid value despite a session error
- src/widgets/__tests__/ApiUsage.localModelFallback.test.ts: output tokens no longer counted in context (upstream sirmalloc#730)
- src/utils/__tests__/renderer-flex-width.test.ts, src/utils/__tests__/renderer-ansi.test.ts, src/utils/__tests__/gradient.test.ts, src/utils/__tests__/renderer-separator-collapse.test.ts, src/utils/__tests__/config.test.ts, src/tui/__tests__/App.test.ts: fork deviation - fork width model (flexMode, 6-column full reserve, kept settings, Terminal Width menu entry)
- src/utils/__tests__/default-colors.test.ts: fork deviation - accept ansi256:/hex: defaults
- src/utils/__tests__/usage-prefetch.test.ts: fork deviation - weekly usage comes from the API, fork widget imports from ApiUsage
- src/widgets/__tests__/CurrentWorkingDir.test.ts, src/widgets/__tests__/CustomCommand.test.ts: Settings literal gains flexMode/compactThreshold
- src/utils/__tests__/context-window.test.ts, src/utils/__tests__/jsonl-metrics.test.ts: restored fork tests upstream dropped
- docs/test-retirements.md: ledger for the dev-reload snapshot-path and RemoteControlStatus glyph (sirmalloc#605) tests upstream replaced
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.

remote-control-status: the "off" status character renders broken

2 participants