Skip to content

Fix helicopter intro rendering on wide aspect ratio window tile settings - #135

Merged
segrax merged 1 commit into
OpenFodder:masterfrom
Forgen:master
Oct 30, 2026
Merged

segrax merged 1 commit into
OpenFodder:masterfrom
Forgen:master

Conversation

@Forgen

@Forgen Forgen commented Oct 29, 2026

Copy link
Copy Markdown
Contributor

Adjust the helicopter intro to render at the original 320-pixel frame when the gameplay surface is wider due to non default --columns setting. The change caps the Amiga scanline loop to the authored width, converts fixed 352-pixel offsets to the current surface width in the PC blits, and restores the gameplay window immediately after the intro so the gameplay presentation is at the requested tile column/row setting.

Example of corruption of intro screen on current production release version 5EB4136 (Dec 16 2025) with columns=28, rows=16:
Screenshot 2026-10-28 235342

(I notice this repo has basically zero inline comments... feel free to remove these if that's policy.)

Adjust the helicopter intro to render at the original 320-pixel frame when the gameplay surface is wider due to --columns or higher resolutions. The change caps the Amiga scanline loop to the authored width, converts fixed 352-pixel offsets to the current surface width in the PC blits, and restores the gameplay window immediately after the intro so the presentation is not stretched or left oversized.
@Forgen

Forgen commented Oct 29, 2026

Copy link
Copy Markdown
Contributor Author

Figured I'd show the "after" screenshots as well:
Screenshot 2026-10-29 000217
Screenshot 2026-10-29 000248

@segrax

segrax commented Oct 30, 2026

Copy link
Copy Markdown
Member

Thanks!

I remember seeing this years ago and never bothered to fix it because no-one ever really used the extra columns mode :D

@segrax segrax self-assigned this Oct 30, 2026
@segrax
segrax merged commit 0df2217 into OpenFodder:master Oct 30, 2026
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