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: progress bar not visible #793

Merged
merged 6 commits into from
Dec 3, 2023

Conversation

punxaphil
Copy link
Contributor

fixes #784

Before

image

After

Playing
image
Paused
image

Copy link
Collaborator

@Drafteed Drafteed left a 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.

src/components/progress.js Outdated Show resolved Hide resolved
src/components/progress.js Outdated Show resolved Hide resolved
punxaphil and others added 2 commits December 3, 2023 07:29
Co-authored-by: Artem Draft <Drafteed@users.noreply.github.com>
Co-authored-by: Artem Draft <Drafteed@users.noreply.github.com>
@punxaphil
Copy link
Contributor Author

Looks very good 👍 Just a small comment about double quotes.

Thanks! updated

Copy link
Collaborator

@Drafteed Drafteed left a comment

Choose a reason for hiding this comment

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

We also should add a backdrop fill, as it was before:

You can add this after .mmp-progress:

.mmp-progress:before {
  content: '';
  position: absolute;
  left: 0;
  right: 0;
  bottom: 0;
  height: var(--mmp-progress-height);
  background-color: rgba(100,100,100,.15);
}

src/components/progress.js Outdated Show resolved Hide resolved
src/components/progress.js Outdated Show resolved Hide resolved
punxaphil and others added 3 commits December 3, 2023 14:58
Co-authored-by: Artem Draft <Drafteed@users.noreply.github.com>
Co-authored-by: Artem Draft <Drafteed@users.noreply.github.com>
@punxaphil
Copy link
Contributor Author

We also should add a backdrop fill, as it was before:

You can add this after .mmp-progress:

.mmp-progress:before {

  content: '';

  position: absolute;

  left: 0;

  right: 0;

  bottom: 0;

  height: var(--mmp-progress-height);

  background-color: rgba(100,100,100,.15);

}

Thanks! Should be fixed now🙂

Copy link
Collaborator

@Drafteed Drafteed left a comment

Choose a reason for hiding this comment

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

Thanks, @johanfrick 👍

@Drafteed Drafteed merged commit b27e36e into kalkih:dev Dec 3, 2023
1 check passed
@punxaphil
Copy link
Contributor Author

Thank you🙂
what's the process of getting the bug fix out to users?

@alexanderznamensky
Copy link

alexanderznamensky commented Dec 4, 2023

Is it possible to install dev version of media player card?
I didn't find such option in HACS.
Or probably we can ask just updated js file?

Anyhow - thankas indeed for your nice contribution !

@punxaphil
Copy link
Contributor Author

@Drafteed

@Drafteed
Copy link
Collaborator

Drafteed commented Dec 4, 2023

I'm planning to make a new release today.

@jhemak
Copy link

jhemak commented Dec 4, 2023

@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.

screenshot

@punxaphil
Copy link
Contributor Author

@jhemak I did the fix, so I should probably take the blame 😉
I can't reproduce what you are showing. What does it mean to be "using this with Mushroom"?

@jhemak
Copy link

jhemak commented Dec 5, 2023

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?

@punxaphil
Copy link
Contributor Author

Not really, still not seeing it with full-cover.

@jhemak
Copy link

jhemak commented Dec 5, 2023

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:

  • With full-cover, there is a gray gradient rectangle at the bottom of the artwork that does not have the same radius in the corners as the artwork itself (it's square, not curved). This is what's peeking thru the progress bar.
  • With cover, there is transparent gray across the entire image, but the corners all reflect the expected radius and the progress bar therefore looks as expected.
  • With cover1 (an invalid value but one I specified for testing), there is no gradient and the progress bar looks as expected.

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!

Picture1

@punxaphil
Copy link
Contributor Author

Alright.
you can always open a separate issue for that.

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.

None yet

4 participants