Adjust dashboard slot scroll sizing (remove inner max-height clamp) #667
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.
Motivation
body_heightso nested content is not truncated or overlapped.Description
src/dashboard/dashboard.rsinsiderender_scrollable_widgetthe innerui.set_max_height(body_height)was removed so the inner UI can compute its natural height.viewport(slot_clip.intersect(viewport)) and applied insideshow_viewportso clipping affects only the viewport.ScrollArea::max_height(body_height)to keep the viewport height fixed, andui.set_min_height(body_height)is retained on the inner UI to preserve minimum sizing.render_scrollable_widget/show_viewportbehavior.Testing
ScrollArea(e.g.todo,clipboard_recent,clipboard_snippets,recent_notes,notes_recent,process_list,query_list,tempfiles) and no code edits were required to avoid truncation after this change.Codex Task