@fhero2030
@SemihRodoplu
Submitted
It seems alike but I am not convinced . I struggled to make button text color different for each column therefore I assigned id attribute for each element . I know something wrong but I could not figure out . Please feel free to make any comment . Thanks
@fhero2030
Posted
It looks good to me. If you want any improvement I think having more padding inside the card should make the difference but the mobile responsiveness is not there. This can be solved if you apply layout using a flex or grid.
@dustinsoos
Submitted
So for me this project was about knocking off a good amount of rust and getting back into challenging myself with the use of CSS. The most difficult part of the project for me was implementing the grid so that the image went to the left of the card on the desktop version.
The areas of code I'm unsure of would just be the set ups of aligning the prices/making sure the picture moved over for the desktop version, and potentially the width on mobile version being a little to small.
A question I would have is when placing all my components in a container I just wanted my full container to take up a little more space in mobile version, what would be a good way to make that happen?
@fhero2030
Posted
I think you should make the width of the add to cart button to 100% and add just a little bit of space to left of card. Also improve the spacing of "PERFUME" you can do that using "letter-spacing: {pixel}" and also adjust font title and price of the card. I also look into your css and it looks like all font in the design have the same family so you should be able to fix that by moving the font-family into the body element.
Marked as helpful
