Skip to content

clip the supernova whiteout to the letterbox#7493

Merged
Goober5000 merged 2 commits into
scp-fs2open:masterfrom
Goober5000:fix/supernova_clip
Jun 2, 2026
Merged

clip the supernova whiteout to the letterbox#7493
Goober5000 merged 2 commits into
scp-fs2open:masterfrom
Goober5000:fix/supernova_clip

Conversation

@Goober5000

@Goober5000 Goober5000 commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

game_render_hud calls gr_reset_clip() before game_flash_diminish, so the sunspot palette flash was painting over the black bars during the supernova camera cut. Re-apply the letterbox clip around gr_flash when it would otherwise overdraw the bars (supernova HIT+ or GM_DEAD), matching the condition already used in game_set_view_clip.

Partial fix for #4152. Tested on the retail Apocalypse mission.

game_render_hud calls gr_reset_clip() before game_flash_diminish, so the
sunspot palette flash was painting over the black bars during the
supernova camera cut. Re-apply the letterbox clip around gr_flash when
it would otherwise overdraw the bars (supernova HIT+ or GM_DEAD),
matching the condition already used in game_set_view_clip.

Partial fix for scp-fs2open#4152.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Goober5000 Goober5000 added the fix A fix for bugs, not-a-bugs, and/or regressions. label Jun 2, 2026
Comment thread freespace2/freespace.cpp Outdated
Per @BMagnu's review feedback on scp-fs2open#7493: factor the yborder calculation
and clip-rect application into two file-local helpers so future changes
to the letterbox proportions live in one place.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Goober5000 Goober5000 merged commit 8114a4c into scp-fs2open:master Jun 2, 2026
20 checks passed
@Goober5000 Goober5000 deleted the fix/supernova_clip branch June 2, 2026 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix A fix for bugs, not-a-bugs, and/or regressions.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants