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

Dark border around VU-Meter GLGS widgets in 200 % scale #11600

Closed
daschuer opened this issue May 29, 2023 · 10 comments
Closed

Dark border around VU-Meter GLGS widgets in 200 % scale #11600

daschuer opened this issue May 29, 2023 · 10 comments

Comments

@daschuer
Copy link
Member

daschuer commented May 29, 2023

Bug Description

Since #10989 the VU-Meter widgets have a black border.

grafik

grafik

grafik
grafik

Version

2.4-alpha

OS

Ubuntu Focal

@daschuer
Copy link
Member Author

@m0dB do you have an idea? Maybe it is related to the issue that the widget thinks it is painted on black and we see antialiasing.

@m0dB
Copy link
Contributor

m0dB commented May 30, 2023

@daschuer it was something silly, in src/util/texture.cpp I was using

pTexture->setWrapMode(QOpenGLTexture::ClampToBorder);

instead of

pTexture->setWrapMode(QOpenGLTexture::ClampToEdge);

What is the correct procedure to create a PR for a specific issue? Do I give the branch a specific name? Do I just mention the issue in the description?

@ronso0
Copy link
Member

ronso0 commented May 30, 2023

Do I just mention the issue in the description?

👍

Do I give the branch a specific name?

Whatever works best for you, though I find that handy while collaborating on a certain branch, it's just easier to remember and easier to spot in a git branch list.

@daschuer
Copy link
Member Author

I can confirm that the suggested change fixes the issue. Thank you.

@daschuer
Copy link
Member Author

I have added the fix here: #11601

@JoergAtGithub JoergAtGithub added this to the 2.4.0 milestone Jun 27, 2023
@JoergAtGithub
Copy link
Member

Did #11722 fix this?

@daschuer
Copy link
Member Author

Yes, I can confirm it is fixed:
grafik

@daschuer
Copy link
Member Author

This one is also back on Ubuntu Jammy :-(

image

@daschuer daschuer reopened this Sep 29, 2023
@m0dB
Copy link
Contributor

m0dB commented Sep 29, 2023

Strange. I'll look into it.

@daschuer
Copy link
Member Author

The fix already lives in #11601, I will verify and reopen it.

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

No branches or pull requests

4 participants