-
-
Notifications
You must be signed in to change notification settings - Fork 212
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: progress bar not visible #793
Conversation
6a045e4
to
7cbfcf8
Compare
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.
Looks very good 👍 Just a small comment about double quotes.
Co-authored-by: Artem Draft <Drafteed@users.noreply.github.com>
Co-authored-by: Artem Draft <Drafteed@users.noreply.github.com>
Thanks! updated |
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.
Co-authored-by: Artem Draft <Drafteed@users.noreply.github.com>
Co-authored-by: Artem Draft <Drafteed@users.noreply.github.com>
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.
Thanks, @johanfrick 👍
Thank you🙂 |
Is it possible to install dev version of media player card? Anyhow - thankas indeed for your nice contribution ! |
I'm planning to make a new release today. |
@Drafteed - Thank you for working to fix this! Is there some small issue still with the rounded corner, at least when using this with Mushroom? This is more visible in light mode than dark mode. |
@jhemak I did the fix, so I should probably take the blame 😉 |
No problem! I just mean that I had Mushroom installed and wasn't sure if that was impacting the formatting in some way. I noticed the issue I found appears only when Artwork is set to full-cover. For all other Artwork options, it displays without that black corner. Does that help? |
Not really, still not seeing it with full-cover. |
Sorry, my mistake. It looks like the issue was not caused by the progress bar fix as I see it present in version 1.16.5 as well. It just wasn't something I noticed until the progress bar was fixed. Here's what I am seeing:
I wish I were smart enough to fix the problem with the full-cover option... in the meantime, I am just using "cover1" as my artwork option since it somehow makes the display look correct for my purpose. Thanks for fixing the main issue with the progress bar! |
Alright. |
fixes #784
Before
After
Playing
Paused