Skip to content

Add: [NewGRF] Station/roadstop animation-triggers 'tile loop' (bit 7) and 'path reservation' (bit 8). - #14080

Merged
frosch123 merged 1 commit into
OpenTTD:masterfrom
frosch123:station_animtriggers
Apr 26, 2025
Merged

frosch123 merged 1 commit into
OpenTTD:masterfrom
frosch123:station_animtriggers

Conversation

@frosch123

@frosch123 frosch123 commented Apr 22, 2025

Copy link
Copy Markdown
Member

Motivation / Problem

  • Houses, industries, objects and airports have a "tile loop" animation trigger. Stations and roadstops do not.
  • Stations have a "path reservation" randomisation trigger, but no animation trigger.

Description

This includes/depends on #14076.

  • Add station/roadstop animation trigger "tile loop" (bit 7).
  • Add station animation trigger "path reservation" (bit 8).

Limitations

Checklist for review

Some things are not automated, and forgotten often. This list is a reminder for the reviewers.

  • The bug fix is important enough to be backported? (label: 'backport requested')
  • This PR touches english.txt or translations? Check the guidelines
  • This PR affects the GS/AI API? (label 'needs review: Script API')
    • ai_changelog.hpp, game_changelog.hpp need updating.
    • The compatibility wrappers (compat_*.nut) need updating.
  • This PR affects the NewGRF API? (label 'needs review: NewGRF')

@frosch123 frosch123 added the needs review: NewGRF Review requested from a NewGRF expert label Apr 22, 2025
@github-actions github-actions Bot added the work: needs rebase This Pull Request needs a rebase label Apr 24, 2025
@frosch123
frosch123 force-pushed the station_animtriggers branch from 2e90bdf to a10f6fa Compare April 24, 2025 07:26
@github-actions github-actions Bot removed the work: needs rebase This Pull Request needs a rebase label Apr 24, 2025
Comment thread src/newgrf_station.cpp Outdated
@frosch123
frosch123 force-pushed the station_animtriggers branch from a10f6fa to 96a0a39 Compare April 25, 2025 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs review: NewGRF Review requested from a NewGRF expert

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants