gui: improve commit/merge symbols for better cross-terminal rendering#5567
gui: improve commit/merge symbols for better cross-terminal rendering#5567wchou158 wants to merge 2 commits into
Conversation
Replace graph symbols used for commits and merges with alternatives that render more consistently across terminals, especially on Windows Terminal with Nerd Fonts. Previous symbols: - Merge: '⏣' - Commit: '◯' Updated symbols: - Merge: '◉' - Commit: '●' The old glyphs can appear misaligned or visually inconsistent depending on font and terminal emulator. The new symbols have better support and provide improved visual clarity without changing semantics. Tested on: - Windows Terminal (various Nerd Fonts) The change is purely visual and does not affect functionality.
|
I also noticed the misalignment especially on Windows. I did some more research on this (or rather, had Claude do the research for me), and came up with #5573, which I prefer over this one. Review and/or testing welcome. |
Left is your symbols, right is mine, I guess it’s a matter of personal preference. You can close my PR if you want. I think I'll just use my own build. Peace. 
|
That may be true, but it's also a matter of trying to find a solution that is likely to work better for most users. I'm not sure if my research on that is worth anything, but at least I made an attempt.
We may have to make the symbols configurable in the end if there are more users who prefer different ones, but I want to see a few more users requesting it before I add that. |
Replace graph symbols used for commits and merges with alternatives that render more consistently across terminals, especially on Windows Terminal with Nerd Fonts.
Previous symbols:
Updated symbols:
The old glyphs can appear misaligned or visually inconsistent depending on font and terminal emulator. The new symbols have better support and provide improved visual clarity without changing semantics.
Tested on:
The change is purely visual and does not affect functionality.
Before:
After:
PR Description
Please check if the PR fulfills these requirements
go generate ./...)