Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Shader Waveforms have filled lead in/out #12015

Closed
daschuer opened this issue Sep 24, 2023 · 7 comments
Closed

Shader Waveforms have filled lead in/out #12015

daschuer opened this issue Sep 24, 2023 · 7 comments
Labels

Comments

@daschuer
Copy link
Member

Bug Description

Normally we have only the outlines of the triangles. With the GLSL types the triangles are filled. Not a real bug, but maybe it hides another issue:
grafik
grafik

Version

2.4-alpha

OS

Ubuntu

@daschuer daschuer added the bug label Sep 24, 2023
@daschuer
Copy link
Member Author

@m0dB do you have an idea why this is the case?

@JoergAtGithub
Copy link
Member

This is an intended visual improvement. We discussed this in the PR review and decided to keep it this way.

@m0dB
Copy link
Contributor

m0dB commented Sep 24, 2023

To be fair, initially it was laziness, as drawing a filled triangle is much simpler than drawing only the outline. But I found I actually liked it much better, visually, so I kept it like this.

So, to put your mind at ease, @daschuer , it doesn't hide another issue.

@daschuer
Copy link
Member Author

Ok thank you for confirm.

@Holzhaus Holzhaus reopened this Sep 24, 2023
@Holzhaus
Copy link
Member

IMHO waveforms should be consistent, so I'd like to keep this open until the other renders are adapted or the fill is removed.

@m0dB
Copy link
Contributor

m0dB commented Oct 13, 2023

Since @Holzhaus insisted, but also because I thought of an approach that is much nicer implementation-wise (a single triangle drawn into an texture which is then repeated pattern-wise in the shader instead of in c++), I have fixed this is #12100

@m0dB
Copy link
Contributor

m0dB commented Oct 15, 2023

@Holzhaus after some discussion, I went with a slightly different look then legacy, using a semi-transparent fill. So, not 100% "consistent" as you requested, but given the 50% transparency of the fill, and 100% match of the outline, I would say at least 50% consistent ;-)

Check out the PR for some screenshots.

@m0dB m0dB closed this as completed Oct 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants