-
Notifications
You must be signed in to change notification settings - Fork 31
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
Audio for Videos #1490
Audio for Videos #1490
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work!
Pressing the volume up button whilst playing a muted video unmutes the AVQueuePlayer
, but the mute button in the UI doesn't get updated. Maybe we need to watch player.muted
for changes and update muted
if so? Could be tricky though if a view update doesn't get triggered when player.muted
changes.
Mlem/App/Views/Shared/Images/Helpers/AnimationControlLayer.swift
Outdated
Show resolved
Hide resolved
Ooh, nice catch! I've added some observation code that should sort it out. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! It would be nice if videos were paused once they leave the screen. If I unmute a video and scroll away, the audio keeps playing.
Checklist
- closes Audio for Videos #1489
Pull Request Information
Enhances video handling:
Known Issues