fix: live theme/font config reload + pane selection fix#1640
Open
flukejones wants to merge 5 commits into
Open
fix: live theme/font config reload + pane selection fix#1640flukejones wants to merge 5 commits into
flukejones wants to merge 5 commits into
Conversation
flukejones
force-pushed
the
main
branch
3 times, most recently
from
b763fcb to
1e5b342
Compare
- focus-switch click skipped on_left_click, kept old anchor - in-flight LMB drag-extended it across pane - clear selection, reset click_state, return on switch Closes raphamorim#1638
font_id-keyed caches survived a font-library swap (ids reused), so line height, spacing, glyph bitmaps stayed stale. - add GridGlyphRasterizer::reset_font_caches - add clear_glyph_cache per grid backend + clear() on vulkan/cpu atlases - call both per panel in update_config on font-library update Closes raphamorim#1639
- update_config reset renderable_content but left panes not-dirty - resize_all_contexts repaints only via PTY wakeup; idle panes never get it - mark per-pane Full damage after the reset
OSC 4/10/11 overrides outlived the app that set them, reverting a pane to the old theme on editor reopen. - clear pane OSC overrides on alt-screen exit - running apps keep overrides across theme change - push only bg reset to frontend
Apps had to restart to pick up a theme change. Implement DECSET 2031 so subscribed apps (helix, neovim) re-query colors and repaint live. - DECSET 2031 opts in to notifications - push unsolicited CSI ? 997 ; <1 dark|2 light> n on theme change - answer CSI ? 996 n query with stored scheme - scheme from OS appearance / [adaptive-theme], threaded via update_config
flukejones
changed the title
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
1 participant
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.
What
Closes
Test
cargo check -p rioterm(metal + cpu): cleancargo check -p sugarloaf --features wgpu: clean#[cfg(target_os = "linux")]; mirrors the metal/wgpu pattern,not natively checked on macOS