@jpyepez
Erik Tangvik
@Etang131All comments
- @Etang131
To fix the background image on the thumb you might want to use: background-image: url("/images/icon-slider.svg"); background-repeat: no-repeat; background-position: center; background-size: auto;
Also how did you get your slider to change colors?
- @sansk@Etang131
It looks like it matches the spec to me. I would consider making the media query change happen a bit later from mobile to desktop though.
- @isaacgalvan10@Etang131
The biggest issue is that if you don't submit an email address your code doesn't allow for the user to change that. Other than that it's pretty close to the design.
- @jessicaary@Etang131
It's really close to the design. On the Why us section I would space those out a tad more. Also I think the font weight is a bit different.
- @clebsjr@Etang131
Great job on your first project.
- @jenny07007@Etang131
Your information bar looks better than the design. However I would make the search bar and search bar font a bit bigger.
- @Nard00@Etang131
I think the only thing is the font weight that is off. Everything else looks like it's correct in desktop mode.
- @NobleSalt@Etang131
You need to make the width quite a bit smaller. Also add a <br> tag after the word reviews and set it as display none in mobile.
