mixxxdj / mixxx

Mixxx is Free DJ software that gives you everything you need to perform live mixes.
http://mixxx.org
Other
4.48k stars 1.27k forks source link

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

Closed daschuer closed 1 year ago

daschuer commented 1 year ago

Bug Description

Since https://github.com/mixxxdj/mixxx/pull/10989 the VU-Meter widgets have a black border.

grafik

grafik

grafik grafik

Version

2.4-alpha

OS

Ubuntu Focal

daschuer commented 1 year ago

@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 commented 1 year ago

@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 commented 1 year ago

Do I just mention the issue in the description?

:+1:

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 commented 1 year ago

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

daschuer commented 1 year ago

I have added the fix here: https://github.com/mixxxdj/mixxx/pull/11601

JoergAtGithub commented 1 year ago

Did https://github.com/mixxxdj/mixxx/pull/11722 fix this?

daschuer commented 1 year ago

Yes, I can confirm it is fixed: grafik

daschuer commented 1 year ago

This one is also back on Ubuntu Jammy :-(

image

m0dB commented 1 year ago

Strange. I'll look into it.

daschuer commented 1 year ago

The fix already lives in https://github.com/mixxxdj/mixxx/pull/11601, I will verify and reopen it.