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

fix: show volume control on supported devices #259

Open
wants to merge 17 commits into
base: development
Choose a base branch
from

Conversation

Saelmala
Copy link
Contributor

@Saelmala Saelmala commented Dec 19, 2024

Volume control is not supported on iOS mobile and iPhone, so this PR makes sure those features are disabled for those platform. Volume control on Android devices are supported so that is allowed.

Hotkeys should not show on tablet but they did so I added a check for that as well.

I also noticed some undesired behavior with the volume slider: If you muted the speaker with the button, and then moved the slider, the button made it look like the speaker wasn't muted - but it was. Now I made it so that if the user mutes the speaker with the button, and then moves the slider - the speaker is unmuted.

@Saelmala Saelmala self-assigned this Dec 19, 2024
@Saelmala Saelmala requested review from malmen237 and LucasMaupin and removed request for malmen237 December 19, 2024 12:26
@Saelmala Saelmala requested a review from malmen237 December 19, 2024 14:24
@Saelmala Saelmala changed the title fix: dont show volume controls on tablet fix: show volume control on supported devices Dec 20, 2024
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