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

Background transparent fix #92

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Background transparent fix #92

wants to merge 12 commits into from

Conversation

Zaarrg
Copy link

@Zaarrg Zaarrg commented Dec 24, 2024

  • Fix for background being transparent after shell video was unmounted
  • Moved setBackground to mpv-version
  • This ensures it only triggers once on start of player. It seems like duration also triggers onEnd (unmount) of player causing background to be set back to visible false. See logs below:

Before - "duration" is triggered after player unmount still setting background as transparent:
image

After - Sets background visible false on "mpv-version" which is always triggered only once on start of playback:
image

(Playback ended after error)

- 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
@Ivshti Ivshti requested review from core1024 and jaruba January 16, 2025 07:16
@jaruba
Copy link
Member

jaruba commented Jan 21, 2025

@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

@jaruba
Copy link
Member

jaruba commented Jan 25, 2025

@Zaarrg ping

@Zaarrg
Copy link
Author

Zaarrg commented Jan 25, 2025

Ah thank you for pinging, did not notice. Emailed you.

Regarding the pr, it can be closed and split, thank you for the feedback.
As the transparent background fix is not needed at all with v5 being on webview2 and the other changes can be easily made into their own pr's.

@jaruba
Copy link
Member

jaruba commented Jan 25, 2025

@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) 😄

@jaruba
Copy link
Member

jaruba commented Jan 25, 2025

@Zaarrg i was wrong, i did receive it, i will reply asap

@Zaarrg
Copy link
Author

Zaarrg commented Jan 25, 2025

@jaruba Glad it came through! Just confirming I used [email protected] as listed on your GitHub. No rush at all—take your time!

@jaruba
Copy link
Member

jaruba commented Jan 25, 2025

@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
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