Skip to content

Fix for #668#669

Open
KenwoodFox wants to merge 1 commit intoedde746:mainfrom
KenwoodFox:fix/no_render_context_fix
Open

Fix for #668#669
KenwoodFox wants to merge 1 commit intoedde746:mainfrom
KenwoodFox:fix/no_render_context_fix

Conversation

@KenwoodFox
Copy link
Copy Markdown

This fixes the issue for me but might not be the right solution, after setVisible i 'think' mpv gets called too early and before drawing we get that 'no render context set' if we wrap a promise around scheduling just one frame it holds it and seems to work for me, since the context is ready by the time it continues.

Maybe that wait could be integrated into the setVisible?

await setVisible(true);

@KenwoodFox KenwoodFox force-pushed the fix/no_render_context_fix branch 2 times, most recently from 50f46be to 96bfc78 Compare March 14, 2026 22:23
@KenwoodFox KenwoodFox force-pushed the fix/no_render_context_fix branch 2 times, most recently from 4825ea1 to 6b248dd Compare March 19, 2026 23:13
@edde746
Copy link
Copy Markdown
Owner

edde746 commented Mar 23, 2026

Wasn't the issue fixed by using the official binary?

@KenwoodFox
Copy link
Copy Markdown
Author

Wasn't the issue fixed by using the official binary?

Kindof! It was fixed with bundling in the official binary but not when built standalone from dart run...

@KenwoodFox KenwoodFox force-pushed the fix/no_render_context_fix branch from 6b248dd to d105788 Compare March 23, 2026 23:37
…r setVisible i 'think' mpv gets called too early and before drawing we get that 'no render context set' if we wrap a promise around scheduling just one frame it holds it and seems to work for me, since the context is ready by the time it continies. Maybe that wait could be integrated into the setvisible?
@KenwoodFox KenwoodFox force-pushed the fix/no_render_context_fix branch from d105788 to fc8d89a Compare April 2, 2026 03:58
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