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

Fix colorized icon with Qt5Compat.GraphicalEffects instead MultiEffet from Qt6 #11308

Merged
merged 1 commit into from
Apr 7, 2024

Conversation

s-lisovenko
Copy link
Contributor

@s-lisovenko s-lisovenko commented Apr 7, 2024

At the moment, MultiEffect has limitations in its operation. It does not work with black images. Therefore, the most logical solution would be to use the features from Qt5

https://forum.qt.io/topic/144070/qt6-color-svg-using-multieffect/8

Multieffect:
Screenshot from 2024-04-07 14-36-26

ColorOverlay:
Screenshot from 2024-04-07 14-35-41

…m Qt6

At the moment, MultiEffect has limitations in its operation.
It does not work with black images. Therefore, the most logical solution would be to use the features from Qt5
@DonLakeFlyer
Copy link
Contributor

@HTRamsey Did you have some other way of fixing this other than reverting back?

@s-lisovenko
Copy link
Contributor Author

s-lisovenko commented Apr 7, 2024

@HTRamsey Did you have some other way of fixing this other than reverting back?

At the moment this is the most reliable way. Moreover, it does not depend on the Qt5Compat library. I see no reason not to continue to use it.

@DonLakeFlyer
Copy link
Contributor

Ah, ok. I thought this was going back to the compat thing. If it works, it works! Thanks.

@DonLakeFlyer DonLakeFlyer merged commit ea2a914 into mavlink:master Apr 7, 2024
10 checks passed
@s-lisovenko
Copy link
Contributor Author

@DonLakeFlyer Sorry, seems like it depends on Qt5Compat. it should be presented in OS. So we need to consider to include it in cmake and qmake

@DonLakeFlyer
Copy link
Contributor

Ok, I'm going to revert since it will fail the AppImage I think with a missing lib.

@s-lisovenko
Copy link
Contributor Author

Ok, I'm going to revert since it will fail the AppImage I think with a missing lib.

Yes, please. For now there no alternative in QT6. Maybe in future releases.

@s-lisovenko
Copy link
Contributor Author

@DonLakeFlyer could we reopen this PR and add missing Qt5Compat dependency. I think we should continue using it as non alternative in qt6.

@DonLakeFlyer
Copy link
Contributor

Doesn't seem like there is a way to reopen a merged PR.

@s-lisovenko
Copy link
Contributor Author

Ok. I will create another with missing library

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

Successfully merging this pull request may close these issues.

2 participants