Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature #21426: Save currently following entity into save file. #21656

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

Harry-Hopkinson
Copy link
Contributor

@Harry-Hopkinson Harry-Hopkinson commented Mar 22, 2024

Implements feature requested in #21426 and saves the currently following entity into the save file. Works with all entities (rollercoasters, peeps etc).

@Harry-Hopkinson Harry-Hopkinson marked this pull request as ready for review March 22, 2024 22:50
@Harry-Hopkinson Harry-Hopkinson force-pushed the save-currently-following-entity branch 2 times, most recently from 50a0590 to f40f0f6 Compare April 2, 2024 13:52
@Harry-Hopkinson Harry-Hopkinson force-pushed the save-currently-following-entity branch 2 times, most recently from ce4abd5 to a9596fd Compare April 7, 2024 07:00
@@ -126,6 +127,7 @@ namespace OpenRCT2
ScreenCoordsXY SavedView;
uint8_t SavedViewRotation;
ZoomLevel SavedViewZoom;
EntityId CurrentlyFollowingEntity{ EntityId::GetNull() };
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Does it matter where this variable is placed? Or should I put it at the bottom?

@Harry-Hopkinson Harry-Hopkinson changed the title Feature #21426: Save followed entity into save file. Feature #21426: Save currently following entity into save file. Apr 17, 2024
@Harry-Hopkinson Harry-Hopkinson force-pushed the save-currently-following-entity branch 3 times, most recently from d5d7b3a to 51d1a59 Compare April 22, 2024 15:46
@Harry-Hopkinson Harry-Hopkinson force-pushed the save-currently-following-entity branch from 51d1a59 to 4fe5aca Compare May 19, 2024 14:50
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.

None yet

1 participant