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

Volume sliders for videos #7429 #9366

Merged
merged 3 commits into from
Nov 27, 2018
Merged

Conversation

sumdog
Copy link
Contributor

@sumdog sumdog commented Nov 27, 2018

This is a patch to add volume support to videos and change the default volume to 50%

Questions:

  • I didn't see any selenium or web browser tests. Are there tests I need to update for this component?
  • I didn't add in Intl/messages for the volume slider since there weren't any for the seek slider. Is that alright or should I add them and to what component?

I've tested this manually on Firefox desktop, Chrome desktop, Firefox mobile and chrome mobile. It matches the behaviour of the seek slider.

Copy link
Member

@Gargron Gargron left a comment

Choose a reason for hiding this comment

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

💯 🎉 👌

@Gargron Gargron merged commit f978afa into mastodon:master Nov 27, 2018
mayaeh pushed a commit to mayaeh/mastodon that referenced this pull request Dec 1, 2018
* Volume sliders for videos mastodon#7429

* Code formatting updates

* Code formatting updates
hiyuki2578 pushed a commit to ProjectMyosotis/mastodon that referenced this pull request Oct 2, 2019
* Volume sliders for videos mastodon#7429

* Code formatting updates

* Code formatting updates
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