Conversation
There was a problem hiding this comment.
Pull request overview
This PR restyles the status panel UI to improve visual layout and clarity. The changes reorganize label positioning, remove redundant information, and swap row order for better visual flow.
- P2P status label moved from right column to left column
- Removed "100 round average" label
- Swapped TPS and Round Time rows for improved visual hierarchy
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| beginning = "" | ||
| end = "" | ||
| middle = strings.Repeat(" ", max(0, size-(lipgloss.Width(beginning)+lipgloss.Width(end)+2))) | ||
| row2 := lipgloss.JoinHorizontal(lipgloss.Left, beginning, middle, end) |
There was a problem hiding this comment.
The calculation of row2's middle spacing occurs before the beginning variable is set with the P2P information. This means row2 will always have the full width as spacing (since both beginning and end are empty strings at this point), and the P2P label won't affect the middle spacing calculation. The middle spacing calculation should be moved after line 109 where the beginning variable is properly set.
There was a problem hiding this comment.
row2 is supposed to be blank, so this is fine. But the same beginning, middle, and end variables are reused for every row, so I can see how it got confused.
| tps = "--" | ||
| } | ||
| beginning = style.Blue.Render(" Round time: ") + roundTime | ||
| beginning = style.Blue.Render(" TPS: ") + tps |
There was a problem hiding this comment.
I'm never going to not see blue spaces before the TPS, lol
There was a problem hiding this comment.
I say that, but they're not blue?!
There was a problem hiding this comment.
They're blue. Daba dee daba day
nullun
left a comment
nullun
left a comment
There was a problem hiding this comment.
This looks much better! Thank you.
2 participants
ℹ Overview
Restyles the top (left) Status panel.
Before
After
✅ Acceptance: