Skip to content

Start writing the v26.05 dev log#52

Draft
AaronVanGeffen wants to merge 1 commit into
mainfrom
v26.05
Draft

Start writing the v26.05 dev log#52
AaronVanGeffen wants to merge 1 commit into
mainfrom
v26.05

Conversation

@AaronVanGeffen
Copy link
Copy Markdown
Member

No description provided.

@AaronVanGeffen
Copy link
Copy Markdown
Member Author

@ZehMatt I think we should add a section on your save state work as well (OpenLoco/OpenLoco#3737). Do you feel like writing something yourself? Feel free to push.

item removal. Suppose you deleted a vehicle while the vehicle list was open? Yeah, the game would crash.
Oops. The same thing happened with the other list windows: stations, towns, industries, and companies.
We've reworked the way items are removed ([#3727](https://github.com/OpenLoco/OpenLoco/pull/3727)),
so this shouldn't bus us again.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

bus

Our rework presented another edge case when regenerating the landscape in the scenario editor.
If you happened to have the town or industry list windows open, their lists would not fully refresh.
Suppose you regenerated with more or fewer towns or indeed industries, the list would refresh,
but not repopulate. This meant entries would potentially be missing -- or lead to invalid entries.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Double --? I'd use single -

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.

3 participants