Skip to content

Increase settings window size to fix clipped content with non-English languages#5064

Open
DateFTP wants to merge 6 commits into
multitheftauto:masterfrom
DateFTP:fix/settings-window-size
Open

Increase settings window size to fix clipped content with non-English languages#5064
DateFTP wants to merge 6 commits into
multitheftauto:masterfrom
DateFTP:fix/settings-window-size

Conversation

@DateFTP

@DateFTP DateFTP commented Jul 18, 2026

Copy link
Copy Markdown

Description

Increases the settings window content size from 680x480 to 720x520 to prevent content clipping, particularly affecting the Advanced tab when using non-English languages.

Problem

The Advanced tab uses IndentX computed from the max text extent of translated labels. With non-English languages (e.g. Russian, German), translated strings are significantly longer, causing combo boxes and other content to overflow past the right edge of the tab panel.

Changes

  • Client/core/CSettings.cpp: Increase kSettingsContentWidth from 680 to 720, kSettingsBaseContentHeight from 480 to 520.

The window remains centered on screen (position is computed as (resolution - windowSize) / 2).

Testing

  • Builds successfully with 0 errors
  • Window is properly centered at all common resolutions
  • Advanced tab content no longer clips with longer localized label text

Checklist

  • Your code should follow the coding guidelines.
  • Smaller pull requests are easier to review. If your pull request is beefy, your pull request should be reviewable commit-by-commit.

DateFTP and others added 6 commits July 19, 2026 00:38
Increase the settings window content size from 680x480 to 720x520
to prevent content clipping in the Advanced tab when using
non-English languages with longer translated labels.
- Compute fComboWidth dynamically based on remaining tab panel width
  instead of hardcoded 170px, clamped between 100-400px
- Fix description label width to use tabPanelSize.fX instead of hardcoded 500px
- Fix 'Check for update now' button position to clamp within tab bounds
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