Skip to content

fix(ui): eliminate horizontal scrollbar overflow with responsive ScrollView bindings - #52

Merged
YoungSx merged 1 commit into
mainfrom
fix/ui-horizontal-scrollbar-overflow
Oct 17, 2026
Merged

YoungSx merged 1 commit into
mainfrom
fix/ui-horizontal-scrollbar-overflow

Conversation

@YoungSx

@YoungSx YoungSx commented Oct 17, 2026

Copy link
Copy Markdown
Contributor

Problem

In the default window size (640x780), the UI panel suffered from an unwanted horizontal scrollbar across the main panel and in modal dialogs. This regression was caused by:

  1. ScrollView not having viewport width constraints, allowing children (like long provider example strings and status notes) to report unconstrained preferred widths that exceeded the 640px window width.
  2. ScrollView lacking horizontal scrollbar policy (ScrollBarPolicy.always-off).
  3. Help tooltips and modal preview texts lacking word wrapping for narrow containers.

Solution

  1. Applied idiomatic Slint ScrollView responsive layout pattern to ScrollView in ui/panel.slint:
    • content-width: sv.visible-width;
    • horizontal-scrollbar-policy: ScrollBarPolicy.always-off;
    • Constrained inner VerticalLayout width with width: sv.visible-width;.
  2. Applied the same pattern to modal detail preview-sv := ScrollView and bounded text wrapping.
  3. Added wrap: word-wrap; to all tooltip text elements in ui/settings.slint.

Verification

  • Virtual desktop visual smoke test (bun run ui:smoke ui/target/debug/squid-ui) passed cleanly without any horizontal scrollbar.
  • cargo test --manifest-path ui/Cargo.toml passed (12/12).
  • bun test passed (812/812).

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ⚠️ Failed 2026-10-17T07:18:15.902938Z 4e2f74c PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@YoungSx
YoungSx merged commit f3fe053 into main Oct 17, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant