Skip to content

Support for change aspect ratio mode in OpenGL widget#555

Merged
valbok merged 3 commits intovalbok:masterfrom
Kioro404:master
Mar 2, 2026
Merged

Support for change aspect ratio mode in OpenGL widget#555
valbok merged 3 commits intovalbok:masterfrom
Kioro404:master

Conversation

@Kioro404
Copy link
Copy Markdown
Contributor

I added support for the OpenGL widget to change the video's aspect ratio mode, and I also set a black background so that the video doesn't appear transparent when it plays.

src/QtMultimedia Outdated
@@ -0,0 +1 @@
/usr/include/qt6/QtMultimedia/6.10.2/QtMultimedia No newline at end of file
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks this is not needed?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, sorry

@valbok
Copy link
Copy Markdown
Owner

valbok commented Feb 28, 2026

Thanks for the contribution, amazing work!

@Kioro404
Copy link
Copy Markdown
Contributor Author

Kioro404 commented Mar 1, 2026

It's not much, but I hope it can be of help.

@Kioro404 Kioro404 closed this Mar 1, 2026
@valbok
Copy link
Copy Markdown
Owner

valbok commented Mar 1, 2026

why closed?

@Kioro404
Copy link
Copy Markdown
Contributor Author

Kioro404 commented Mar 2, 2026

I didn't have to close it?, it's my first pull request

@Kioro404 Kioro404 reopened this Mar 2, 2026
@valbok valbok merged commit 6e98453 into valbok:master Mar 2, 2026
18 of 28 checks passed
@valbok
Copy link
Copy Markdown
Owner

valbok commented Mar 2, 2026

I didn't have to close it?, it's my first pull request

Thanks and congrats, now you are the contributor https://github.com/valbok/QtAVPlayer/blob/master/CONTRIBUTORS.md

@Kioro404
Copy link
Copy Markdown
Contributor Author

Kioro404 commented Mar 2, 2026

Thanks to you ^^

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