Skip to content
This repository has been archived by the owner on Apr 20, 2023. It is now read-only.

Add depth buffer option #3

Open
joseyose opened this issue Nov 9, 2022 · 4 comments
Open

Add depth buffer option #3

joseyose opened this issue Nov 9, 2022 · 4 comments

Comments

@joseyose
Copy link

joseyose commented Nov 9, 2022

Right now in the example you are only rendering a triangle so there's no need for a depth buffer, but as soon you try to integrate this in application that is depth testing the ui will not draw correctly.

@cohaereo cohaereo closed this as not planned Won't fix, can't repro, duplicate, stale Nov 9, 2022
@cohaereo cohaereo reopened this Nov 9, 2022
@cohaereo
Copy link
Owner

cohaereo commented Nov 9, 2022

Closed this on accident, sorry for that.

I thought that EGUI wouldn't need a depth buffer? I'm guessing they're not doing Z-sorting yet? I personally haven't had any problems with it, so if you could give an example of the lack of depth testing causing graphical issues, that would be great!

@joseyose
Copy link
Author

joseyose commented Nov 9, 2022

No worries!
Here's the example included of just adding depth test and clearing both the color and depth buffer:
image

@cohaereo
Copy link
Owner

Did some testing myself, was able to reproduce that exact same issue.
I saw you already fixed this on your fork, if you can PR your changes I'll update the crate

@joseyose
Copy link
Author

I need to do some more testing, but I'll try and do a PR if it's working as intended.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants