Skip to content

fix(gui): keep Settings Language description on one line#72

Merged
AprilNEA merged 1 commit into
AprilNEA:masterfrom
davidbudnick:fix/settings-language-spacing
Jun 2, 2026
Merged

fix(gui): keep Settings Language description on one line#72
AprilNEA merged 1 commit into
AprilNEA:masterfrom
davidbudnick:fix/settings-language-spacing

Conversation

@davidbudnick
Copy link
Copy Markdown
Contributor

@davidbudnick davidbudnick commented Jun 1, 2026

Fixes #61.

In Settings → Language, the "Choose the interface language." description rendered one character per line (see the issue screenshot). The language Select's root element is size_full, so it claimed the entire row and starved the description column to near-zero width, forcing the text to wrap per-character.

Fix

  • Wrap the language Select in a fixed-size, non-shrinking box (flex_shrink_0().w(220px).h_6()) so it no longer expands to fill the row.
  • Let the settings scroll container flex and scroll (flex_1().min_h(0)) and grow the window height (360 → 460) so the Language section isn't clipped.

Cherry-picked from #24 — the Settings/Language spacing fix only, none of the locale-expansion or asset changes.

cargo check -p openlogi-gui green.

Screenshot 2026-06-01 at 6 31 03 PM

Copy link
Copy Markdown

@pullfrog pullfrog Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ No new issues found.

Reviewed changes — PR #72 fixes per-character wrapping of the "Choose the interface language." description in Settings → Language by pinning the Select's size_full root to a fixed flex_shrink_0 box so it no longer starves the description column.

  • Wrap language Select in fixed-size boxdiv().flex_shrink_0().w(px(220.)).h_6() prevents the size_full root from claiming the entire row and forcing the description into per-character wrapping.
  • Make scroll container flex+scroll.flex_1().min_h(px(0.)) on the inner v_flex enables proper vertical fill and scrolling.
  • Grow window height — 360 → 460 so the Language section isn't clipped.

Pullfrog  | View workflow run | Using Big Pickle (free) (credentials for Anthropic not configured) | 𝕏

@davidbudnick davidbudnick force-pushed the fix/settings-language-spacing branch from 8134311 to dd6b132 Compare June 1, 2026 23:31
@AprilNEA AprilNEA merged commit 2fec0c6 into AprilNEA:master Jun 2, 2026
7 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.

Change Interface Language Spacing Error in settings

2 participants