modus-web-components icon indicating copy to clipboard operation
modus-web-components copied to clipboard

Slider: fix modus slider selection

Open SuruthiShriP opened this issue 1 year ago • 6 comments

Description

There is no difference in the visibility of selected and non selected part of the slider as the entire part is shown in blue color.

https://deploy-preview-3026--moduswebcomponents.netlify.app/?path=/story/components-slider--default

References #2717

SuruthiShriP avatar Nov 25 '24 15:11 SuruthiShriP

Deploy Preview for moduswebcomponents ready!

Built without sensitive environment variables

Name Link
Latest commit 77e4ff787b47e2aa73022382f556a5eaca42d948
Latest deploy log https://app.netlify.com/projects/moduswebcomponents/deploys/68590b93ca25d40008a14eea
Deploy Preview https://deploy-preview-3026--moduswebcomponents.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

Lighthouse
1 paths audited
Performance: 21 (🔴 down 13 from production)
Accessibility: 75 (no change from production)
Best Practices: 92 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

netlify[bot] avatar Nov 25 '24 15:11 netlify[bot]

https://deploy-preview-3026--moduswebcomponents.netlify.app/?path=/story/components-slider--default

This doesn't appear to be working...

coliff avatar Nov 25 '24 15:11 coliff

@SuruthiShriP - Build is failing - please run tests locally to ensure all is working before committing changes. Thanks! image

coliff avatar Nov 25 '24 15:11 coliff

@SuruthiShriP - Build is failing - please run tests locally to ensure all is working before committing changes. Thanks! image

My local tests are running successfully, but not sure why its failing here. Can you please help me?

SuruthiShriP avatar Nov 25 '24 16:11 SuruthiShriP

Yes, the disabled state should be gray. The slider design in this PR is good, an improvement, but it's actually different to what appears on the Figma file (which doesn't have a different color for the selected range).

We need input from: @enowak1031 and @mitch-trimble if we can proceed with this.

image

coliff avatar Nov 26 '24 07:11 coliff

image does this makes sense?

SuruthiShriP avatar Nov 26 '24 07:11 SuruthiShriP