-
Notifications
You must be signed in to change notification settings - Fork 3k
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
vo_gpu: split 'alpha' into two options #9621
Conversation
This comment was marked as off-topic.
This comment was marked as off-topic.
I think I would prefer naming this option something like "--window-transparent", to go with the other "--window-" flags. |
927973d
to
b51903c
Compare
Updated the option name to |
It's because you added the option value to |
Ok indeed. Does the option belong there? I added it to vo_opts since that's where others window- options were defined. |
Setting the |
b51903c
to
b6e18bd
Compare
I put the option in |
Added an |
b6e18bd
to
385f2ee
Compare
a91ff5d
to
d36b557
Compare
DOCS/man/options.rst
Outdated
all platforms. If alpha framebuffers are unavailable, it silently falls | ||
back on a normal framebuffer. Note that if you set the ``--fbo-format`` | ||
option to a non-default value, a format with alpha must be specified, | ||
or this won't work. Whether this really works depends on the windowing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The sentence about --fbo-format
should go into the documentation of --alpha=yes
, since it's relevant only for alpha blending within the pipeline, rather than display onto the screen.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Like this? I'll admit that I don't full understand the workings of fbo-format
, even after trying its different values out.
Currently it covers two somewhat orthogonal concepts: * whether to create a framebuffer with alpha component * how to blend video with alpha information This commit creates a new flag 'window-transparent' which now controls the first part, while 'alpha' now only covers the second. Previously it would only be possible to make the window transparent with 'alpha=yes', which prevented blending of the image with tiles or the background. This split allow using different blending strategies while keeping the window transparent. The special value window-transparent=auto is created for backwards compatibility.
d36b557
to
8e45d0e
Compare
ping |
So if i understand correctly this PR will let users like me to have a video with alpha channel display properly. Now i can not do that on my system. If this PR can do that. I would like that very much. If a way to do that already exist please let me know. Sorry if this comment is not relevant i just searching for a way solve my problem and i find this PR. |
What you're trying to achieve should already be possible, all you have to do is pass Regarding, this PR, I've noticed that it does not work with |
When i try to do the |
Ah yes, it could be that on X11 + mesa this doesn't work, see also the comments in #9862. Can you try on a wayland compositor? Anyway this is not really on-topic for this PR, maybe ask on IRC or create a new issue. |
Superseded by #13493. |
Currently it covers two somewhat orthogonal concepts:
This commit creates a new flag 'fbo-with-alpha' which now controls
the first part, while 'alpha' now only covers the second.
Previously it would only be possible to make the window transparent
with 'alpha=yes', which prevented blending of the image with tiles or
the background.
This split allow using different blending strategies while keeping the
window transparent.
One downside is that only specifying 'alpha=yes' no longer makes sense
without specifying 'fbo-with-alpha' as well.
Fixes #9615
If this seems ok conceptually, I can adjust the docs and document the new flag.
Tested on wayland with wayfire (wlroots-based) and weston.