feat(store-ui): slider

[emersonlaurentino]

What's the purpose of this pull request?

closes #835

How it works?

How to test it?

References

[netlify]

✔️ Deploy Preview for storeui ready!

🔨 Explore the source changes: 3e2f791

🔍 Inspect the deploy log: https://app.netlify.com/sites/storeui/deploys/612cf730028b8900084caa03

😎 Browse the preview: https://deploy-preview-917--storeui.netlify.app

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 3e2f791:

Sandbox	Source
Store UI Typescript	Configuration

[tlgimenes]

Is it possible add these badges?

[emersonlaurentino]

Is it possible add these badges?

No, it's not possible. but I think it's right because the value of the tooltip is the same as the below value (R$ 1.201.56).

It made me think that maybe we should have a Prefix to add the currency sign for example, or maybe a mask to format the value.

[victorhmp]

@emersonlaurentino Not regarding the code itself yet, but I'm not sure about calling this component Slider since we're already using this name for the generic idea of a Carousel/Shelf, that's why there's the useSlider hook.

I had a quick chat with @lariciamota about this when she was looking into developing this, and we came up with the name Range for this component. How do you feel about that?

[emersonlaurentino]

@emersonlaurentino Not regarding the code itself yet, but I'm not sure about calling this component Slider since we're already using this name for the generic idea of a Carousel/Shelf, that's why there's the useSlider hook.

I had a quick chat with @lariciamota about this when she was looking into developing this, and we came up with the name Range for this component. How do you feel about that?

I'm just following the w3 specification called Slider. You can read more about it here.

I think that's the right name and we must rename the useSlider to useCarousel. What do you think?

[victorhmp]

I'm just following the w3 specification called Slider. You can read more about it here.

I think that's the right name and we must rename the useSlider to useCarousel. What do you think?

That's great! I didn't know that w3 had an official name for that, we should definitely follow it!

And sure! I agree, in this case we should rename the hook 😄

[igorbrasileiro]

Remember to add it to stores to see if it changes performance.

[igorbrasileiro]

Great job @emersonlaurentino, nice documentation and component!
I left some comments and found a visual bug

[emersonlaurentino]

@victorhmp @igorbrasileiro I apply your suggestions, can you review again?