Skip to content

DrawOverFov#6530

Open
Pok27 wants to merge 1 commit intospace-wizards:masterfrom
Pok27:DrawOverFov
Open

DrawOverFov#6530
Pok27 wants to merge 1 commit intospace-wizards:masterfrom
Pok27:DrawOverFov

Conversation

@Pok27
Copy link
Copy Markdown
Contributor

@Pok27 Pok27 commented Apr 18, 2026

This PR adds support for rendering selected sprites above FOV via DrawOverFov and a dedicated DrawEntitiesOverFov pass. Sprites marked with DrawOverFov are now skipped during the normal entity pass, collected while preserving the existing draw ordering, and rendered after FOV has been applied so they remain visible on top of it.

This will make effects like smoke look better.

before
image

after
image

@Pok27 Pok27 marked this pull request as draft April 21, 2026 15:27
@Pok27 Pok27 marked this pull request as ready for review April 21, 2026 15:41
@PJB3005
Copy link
Copy Markdown
Member

PJB3005 commented May 5, 2026

This will make effects like smoke look better.

I'm gonna be honest, that after screenshot still looks terrible.

@PJB3005 PJB3005 added T: New Feature Type: New feature A: Rendering Anything related to putting pixels on the screen labels May 5, 2026
@mirrorcult
Copy link
Copy Markdown
Contributor

yeah, im not really sure theres a noticeable difference in quality between the two there

@Pok27
Copy link
Copy Markdown
Contributor Author

Pok27 commented May 5, 2026

The main problem with the smoke right now is the wall shadows, which I think can be turned off in the prototype. But it's still better
image
image

@PJB3005
Copy link
Copy Markdown
Member

PJB3005 commented May 5, 2026

I'm really not a fan of adding any additional complexity into our garbage renderer for a feature that still isn't actually acceptable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A: Rendering Anything related to putting pixels on the screen T: New Feature Type: New feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants