Skip to content

fix: wrap long game location paths#28

Merged
trchitho merged 1 commit into
swstegall:developfrom
trchitho:fix/wrap-game-location
Jun 14, 2026
Merged

fix: wrap long game location paths#28
trchitho merged 1 commit into
swstegall:developfrom
trchitho:fix/wrap-game-location

Conversation

@trchitho

Copy link
Copy Markdown
Collaborator

Summary

  • use egui's wrapped horizontal layout for the game location row
  • keep long paths visible without forcing the launcher window wider

Validation

  • cargo fmt --all -- --check
  • cargo test --locked (30 tests passed)
  • git diff --check

The repository's x86_64 Windows clippy run still reports pre-existing architecture-specific warnings in src/platform/windows.rs; the documented release target is i686 Windows.

Closes #26

Copilot AI review requested due to automatic review settings June 13, 2026 13:42

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates the launcher UI layout to allow the “Game location” row to wrap when horizontal space is constrained.

Changes:

  • Switched the game location row container from horizontal to horizontal_wrapped to improve responsiveness in narrow window sizes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@swstegall

Copy link
Copy Markdown
Owner

Please consult with me before raising a PR if you are not affiliated with the project.

@swstegall swstegall closed this Jun 13, 2026
@swstegall swstegall reopened this Jun 14, 2026
@trchitho
trchitho merged commit b16ca6c into swstegall:develop Jun 14, 2026
8 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.

Launcher: long Game location path overflows the window — enable word wrapping

3 participants