Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add handling for iOS in editor background/audio file selection #30999

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

Conversation

frenzibyte
Copy link
Member

@frenzibyte frenzibyte commented Dec 7, 2024

Split the selection scope dialog and added ISystemFileSelector support:

ScreenRecording_12-18-2024.10-35-52_1.mov

Since this is almost the main usage of file selectors and it's directly using ISystemFileSelector, perhaps we don't really have to embed ISystemFileSelector within FileSelector as done in ppy/osu-framework#6445. Not sure, not relevant to this PR anyway.

@frenzibyte frenzibyte force-pushed the ios-hide-file-selector-popover branch from ebfe417 to 33f3ba9 Compare December 18, 2024 15:43
@pull-request-size pull-request-size bot added size/L and removed size/M labels Dec 18, 2024
@frenzibyte frenzibyte changed the title Hide popover content when selecting background/audio in iOS Add handling for iOS in editor background/audio file selection Dec 18, 2024
@peppy
Copy link
Member

peppy commented Jan 24, 2025

@frenzibyte test failures here look relevant, please have a check.

@frenzibyte
Copy link
Member Author

I cannot repro locally and I found one commit that's outside of this branch which saw the same test failures: https://github.com/ppy/osu/actions/runs/12954965131/job/36138085475. Not entirely sure what the cause is, and I can't repo locally so it's hard for me to investigate.

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants