Skip to content

Fix OvGame not rendering properly#338

Merged
adriengivry merged 2 commits intodevelopfrom
fix/ovgame_framebuffer
Feb 10, 2025
Merged

Fix OvGame not rendering properly#338
adriengivry merged 2 commits intodevelopfrom
fix/ovgame_framebuffer

Conversation

@adriengivry
Copy link
Member

@adriengivry adriengivry commented Nov 29, 2024

Description

OvGame isn't properly rendering the game since post-process effects have been added. It seems like drawing directly to the backbuffer isn't supported anymore, as the renderer relies on an output framebuffer.

This PR adds a framebuffer to OvGame, and implements a solution to blit the framebuffer onto the backbuffer.

Related Issues

Fixes #337
Fixes #331

@adriengivry adriengivry added Bug Graphics Graphical feature labels Nov 29, 2024
@adriengivry adriengivry self-assigned this Nov 29, 2024
@adriengivry adriengivry marked this pull request as draft November 29, 2024 16:48
@adriengivry adriengivry changed the title Fix OvGame not rendering properly [WIP] Fix OvGame not rendering properly Nov 29, 2024
@adriengivry adriengivry force-pushed the fix/ovgame_framebuffer branch from 918c639 to d215a76 Compare February 9, 2025 22:28
@adriengivry adriengivry changed the title [WIP] Fix OvGame not rendering properly Fix OvGame not rendering properly Feb 9, 2025
@adriengivry adriengivry marked this pull request as ready for review February 9, 2025 22:30
@adriengivry
Copy link
Member Author

Ready for review! I ended up adding a BlitToBackBuffer method to the Framebuffer class to blit its content to the backbuffer.

@adriengivry adriengivry added the Critical Something that should be fixed quickly label Feb 9, 2025
@adriengivry adriengivry merged commit f1d859a into develop Feb 10, 2025
1 check passed
@adriengivry adriengivry deleted the fix/ovgame_framebuffer branch February 10, 2025 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Critical Something that should be fixed quickly Graphics Graphical feature

Development

Successfully merging this pull request may close these issues.

OvGame isn't rendering properly OvGame runtime crash since renderer refactor

2 participants

Comments