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

Display storyboard in editor background #31364

Merged
merged 7 commits into from
Jan 10, 2025

Conversation

bdach
Copy link
Collaborator

@bdach bdach commented Dec 31, 2024

Fixes the main part of #31144. Support for selecting a video will come later.

2024-12-31.14-02-31.mp4
2024-12-31.14-03-19.mp4

Making this work was an absolutely awful time full of dealing with delightfully kooky issues, and yielded in a very weird-shaped contraption. There is at least one issue remaining wherein storyboard videos do not actually display until the track is started in editor, but that is 99% a framework issue and I do not currently have the mental fortitude to diagnose further.

Fixes the main part of ppy#31144.
Support for selecting a video will come later.

Making this work was an absolutely awful time full of dealing with
delightfully kooky issues, and yielded in a very weird-shaped
contraption. There is at least one issue remaining wherein storyboard
videos do not actually display until the track is started in editor, but
that is 99% a framework issue and I do not currently have the mental
fortitude to diagnose further.
@bdach bdach force-pushed the editor-storyboard-display-2 branch from c04433a to 78c7ee1 Compare December 31, 2024 14:18
@bdach bdach requested a review from peppy January 8, 2025 07:13
Copy link
Member

@peppy peppy left a comment

Choose a reason for hiding this comment

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

Seems fine except I'm not sure about the background layer colouring

@peppy peppy self-requested a review January 10, 2025 04:56
@peppy peppy merged commit 73adc4d into ppy:master Jan 10, 2025
7 of 10 checks passed
@bdach bdach deleted the editor-storyboard-display-2 branch January 10, 2025 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants