Skip to content

Allow resizing the vehicle list window more flexibly#3531

Merged
AaronVanGeffen merged 10 commits intoOpenLoco:masterfrom
AaronVanGeffen:vehicle-list-resize
Dec 11, 2025
Merged

Allow resizing the vehicle list window more flexibly#3531
AaronVanGeffen merged 10 commits intoOpenLoco:masterfrom
AaronVanGeffen:vehicle-list-resize

Conversation

@AaronVanGeffen
Copy link
Copy Markdown
Member

@AaronVanGeffen AaronVanGeffen commented Dec 10, 2025

After:

Screen.Recording.2025-12-10.at.14.49.25.mp4

Before:

Screen.Recording.2025-12-10.at.14.49.57.mp4

Copy link
Copy Markdown
Contributor

@LeftofZen LeftofZen left a comment

Choose a reason for hiding this comment

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

Few small points:

  • The dropdowns at the bottom of the window are too big/the area they're in is too small; they;re over the window border:
    image
    image

  • Resizing the window smaller makes the dropdowns at the bottom disappear:
    image

  • Resizing the window smaller makes the grid columns disappear (instead of resizing smaller to fit them on:
    image

  • It seems there is a max size with the vehicle window (there was no max size with the object selection window)

  • While this is vanilla behaviour, the text columns rendered over the vehicle list makes them hard to read. Perhaps make the text columns have a translucent background? (is that even possible in loco?). If there is no better solution, happy to leave this for now as it is vanilla behaviour.

image

@auge8472
Copy link
Copy Markdown

auge8472 commented Dec 11, 2025

  • While this is vanilla behaviour, the text columns rendered over the vehicle list makes them hard to read. Perhaps make the text columns have a translucent background? (is that even possible in loco?). If there is no better solution, happy to leave this for now as it is vanilla behaviour.

Looking at the screenshot I would like to propose to make the rows taller to visually separating the text and the image. But that should IMHO be the subject of another PR.

@AaronVanGeffen
Copy link
Copy Markdown
Member Author

Thanks for giving this a go, @LeftofZen! Appreciate the feedback.

The dropdowns at the bottom of the window are too big/the area they're in is too small; they;re over the window border

This is a good point. I haven't changed anything here, but this seems to be a mistake from when I introduced these dropdowns in the first place. I think we can spare a few more pixels 😄

Resizing the window smaller makes the dropdowns at the bottom disappear

This is deliberate, so the dropdowns don't overlap with the status line. I can tweak cut-off point a little more, though.

Resizing the window smaller makes the grid columns disappear (instead of resizing smaller to fit them on

This is true, but vanilla behaviour. Widening the window only increases the size of the name column. Shrinking the columns is a bit difficult tbh, as the texts don't have good breaking points for auto-ellipsism.

It seems there is a max size with the vehicle window (there was no max size with the object selection window)

Yes, all windows have a max size. The object selection window has 2000x2000, while this one has 1200x1200. I chose to go with smaller dimensions due to the 'auto resize' feature being enabled for this one. (Click resize handle to maximise.) Thinking on it some more, though, I think we can increase the vertical limit with no issues, though, so I'll amend that.

While this is vanilla behaviour, the text columns rendered over the vehicle list makes them hard to read. Perhaps make the text columns have a translucent background? (is that even possible in loco?). If there is no better solution, happy to leave this for now as it is vanilla behaviour.

This doesn't sound like something we can do easily, but I'm open to suggestions. Perhaps you could file an issue with a mockup proposal? It's a bit out of scope for this PR, imo.

@AaronVanGeffen AaronVanGeffen added this to the v25.12 milestone Dec 11, 2025
@AaronVanGeffen AaronVanGeffen merged commit 580470a into OpenLoco:master Dec 11, 2025
14 checks passed
@AaronVanGeffen AaronVanGeffen deleted the vehicle-list-resize branch December 11, 2025 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog Requires a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Maximising Vehicle List window interferes with filter drop-downs and vice versa

4 participants