-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
feat: Add volume icon and percent label (#2) #55
Conversation
Someone is attempting to deploy a commit to a Personal Account owned by @orhun on Vercel. @orhun first needs to authorize it. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Hmm, I'm not sure how that would look and what's the best design to choose here. Can you possibly share a mockup of that?
Definitely an intriguing idea + looks pretty good. ping @WKFO for comment. |
First of all, good idea. I think the volume icon should be there; because on the player, there is an indication as to what is what, except the volume control. If you don't want crowd, there is the small player, whose use I don't really see to be honest. Circular slider looks cooler but may be slightly more cumbersome to use; because a lot of people can quickly tab in, aim at a straight slider and tab back out, but aiming the correct position on a circular slider may require more effort. |
Move percent label on the top of slider
Yeah, it looks definitely better with the last update, thanks @Cthulhu2! Is there anything else needed to be done about this? |
Can I get a final review on this @WKFO? |
Works seamlessly, thanks a lot! @Cthulhu2 |
No description provided.