@pikapikamart
Posted
Good work on this one. But a really small issue, there is an appearance of scrollbar at the bottom and it was cause by the first container of your image, or the container of your slider. To fix this, just add a width: 100%
to it so that, your declared overflow
will be applied and I don't know if you are supposed to display to slider at once? Since right now, there is the top slider and bottom slider appearance, is that what the challenge says? But still, slider functions well^
1
@astroud
Posted
@pikamart Thanks for the feedback Pikamart. Sorry for the late reply, been under the weather.
What browser are you seeing that scrollbar in? I can't replicate it on my Mac with chrome/safari/firefox or on my iPhone.
Looks like my enhanced submission would benefit from some explanatory text in between the slider components.
The component defaults to the small size, but you can also pass a prop of large=true to get the full size (which is responsive).
I figured including both on the same page makes a better demo. I placed the large component at the top for the design comparison, but it sounds like the page would benefit from some clarifying text between the sliders.
0