-
Notifications
You must be signed in to change notification settings - Fork 10
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
Background transparent fix #92
base: master
Are you sure you want to change the base?
Conversation
- Moved setBackground to mpv-version - This ensures it only triggers once on start of player. It seems like duration also triggers onEnd of player.
- Check if webview or qt to prevent setting opengl-cb for webview app
@Zaarrg this PR will probably need to be split into multiple PRs as it handles more things than just the background fix, you're doing some amazing work and we congratulate you for that, i would suggest that you send me an email (my email is publicly available on my github page) to discuss if you want to work on the official Stremio shells, and also discuss the use of our trademarked brand in your unofficial shell because while we allow the use of our brand by some community projects, we only do so based on acceptance of the use case and the way it is presented, your project breaks quite a few of our internal rules regarding allowing the use of the brand, i'm sure we can figure it out but we'd need to discuss it properly |
@Zaarrg ping |
Ah thank you for pinging, did not notice. Emailed you. Regarding the pr, it can be closed and split, thank you for the feedback. |
@Zaarrg are you sure that you wrote the right email? I didn't receive anything yet and even tested by sending myself an email (and that worked) 😄 |
@Zaarrg i was wrong, i did receive it, i will reply asap |
@jaruba Glad it came through! Just confirming I used [email protected] as listed on your GitHub. No rush at all—take your time! |
@Zaarrg Ivo will answer and CC me, he found ur response too now, he is the original developer of Stremio and the best person to talk to about both the shell and assessing the possibility of joining the official team |
- Fixed setBackground running after unload - setBackground(false); has to happen in duration otherwise blackscreen issue see [here](Zaarrg/stremio-desktop-v5#12). - Hopefully this does not cause the blackscreen issue again but it shouldnt.
- Sets gpu-next directly and correct input props
- Support for subtitle delay for embedded subtitles in shell player
Before -
"duration"
is triggered after player unmount still setting background as transparent:After - Sets background visible false on
"mpv-version"
which is always triggered only once on start of playback:(Playback ended after error)