devs/cproof
1
2
Fork 1
Code Issues 43 Pull Requests 6 Actions Packages Projects 1 Releases Wiki Activity

fix(ui): reserve pad height per message to prevent multi-line clip and scroll desync #136

Manually merged
jabber.developer2 merged 1 commits from fix/multiline-pad-clip into master 2026-06-20 09:41:17 +00:00
Conversation 2 Commits 1 Files Changed 1 +1 -4
1 changed files with 1 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/ui/window.c
View File

@@ -72,10 +72,7 @@ _win_ensure_pad_capacity(ProfWin* window, WINDOW* win, int lines_needed)
static int
_win_estimated_lines(WINDOW* win, const char* const message, int indent, int pad_indent)
{
int usable = getmaxx(win) - (indent + pad_indent);
if (usable < 1) {
usable = 1;
}
int usable = MAX(1, getmaxx(win) - (indent + pad_indent));
int newlines = 0;
jabber.developer marked this conversation as resolved Outdated
jabber.developer commented 2026-06-20 09:18:32 +00:00
Outdated
Review
Copy Link

We might also use MAX(1, ...) instead, but this variant is also fine.

We might also use `MAX(1, ...)` instead, but this variant is also fine.
jabber.developer2 commented 2026-06-20 09:25:54 +00:00
Outdated
Review
Copy Link

Agreed, used MAX instead

Agreed, used MAX instead
for (const char* p = message; *p; p++) {
if (*p == '\n') {