Skip to content

Bundle Nerd Font Symbols fallback - #2

Closed
estem8 wants to merge 3 commits into
mainfrom
fix/bundle-nerd-font-symbols
Closed

estem8 wants to merge 3 commits into
mainfrom
fix/bundle-nerd-font-symbols

Conversation

@estem8

@estem8 estem8 commented Nov 4, 2026 •

Copy link
Copy Markdown
Owner

Session titles can contain Nerd Font private-use glyphs such as U+F082B (md-image_off). System fallback fonts may not include these glyphs, so OmpDeck shows a tofu box. Bundle Nerd Font Symbols in the Proportional fallback chain so these glyphs render without a user-installed font.

Verified locally: cargo test -p omp-deck ui_fonts::tests::nerd_font_private_use_glyphs_resolve_through_proportional_fallback; cargo build -p omp-deck; visual check in WSL.

lavinhoque33 and others added 3 commits November 3, 2026 18:49
… narrower tab-area bar: segment priority is now context, usage, git, cost, tokens, with the path chip elastic (elided from the front into the leftover width, full path on hover/copy) and the model chip yielding toward its minimum; tests that executed a freshly written fake script now install it through another process, so a sibling test's fork can no longer make exec fail with ETXTBSY.
…e or phone no longer asks to discard the composer draft: with live control the ctl runs exactly what omp's /switch and /effort call (resolveCliModel + setModelTemporary, resolveThinkingArgument + setThinkingLevel) and leaves the draft, caret, pastes and images untouched; a switch that needs compaction first, a tab without live control, or omp internals that drifted (answered as unsupported) fall back to the typed command. The bar's model follows the selected model after a mid-turn switch, chat replies keep the model they were written under, and successful direct changes show omp's own message instead of a false error toast.
@coderabbitai

coderabbitai Bot commented Nov 4, 2026

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 2a3f4595-9087-4254-916e-3a1528d4a024
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@estem8

estem8 commented Nov 4, 2026

Copy link
Copy Markdown
Owner Author

Superseded by the correctly targeted upstream pull request: lavinhoque33#1. The fork's main branch was behind upstream and made this internal PR show unrelated changes.

@estem8 estem8 closed this Nov 4, 2026
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