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

Add moods (times of day) to Truck Town demo #1138

Merged
merged 1 commit into from
Nov 25, 2024

Conversation

Calinou
Copy link
Member

@Calinou Calinou commented Nov 22, 2024

Four moods are available: sunrise, day (default), sunset, night. Sunset and night also feature artificial point lights placed in the town scene.

The mood can be set on the main menu before playing. It can also be cycled during gameplay by pressing M or D-Pad Down.

Tested in all rendering methods. Note that it'll look better in Compatibility once #1132 is merged.

Preview

Sunrise

truck_town_sunrise.mp4

Day

Same appearance as before.

Sunset

truck_town_sunset.mp4

Night

truck_town_night.mp4

@aaronfranke
Copy link
Member

aaronfranke commented Nov 22, 2024

We may want to merge #1110 first which includes a bunch of fixes. EDIT: Done.

Four moods are available: sunrise, day, sunset, night. Sunset and night
also feature artificial point lights placed in the town scene.

The mood can be set on the main menu before playing. It can also be cycled
during gameplay by pressing M or D-Pad Down.
@Calinou
Copy link
Member Author

Calinou commented Nov 25, 2024

Rebased and tested again, it works as expected.

@aaronfranke
Copy link
Member

Tested and looked over the changes, looks great! The only thing I'm not sure about is, since this is a substantial and aesthetic change (as opposed to a small bugfix), I would not personally cherrypick this.

@aaronfranke aaronfranke merged commit 51a7d38 into godotengine:master Nov 25, 2024
1 check passed
@Calinou Calinou deleted the truck-town-add-moods branch November 25, 2024 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants