Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Mobile_first_Social proof section

@snake321

Desktop design screenshot for the Social proof section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


hi guys finally I have completed social Proof section plz give your feedback on my rating section and my stars images is there any better way to specify them? and how to give the margin on background image

Community feedback

@GrzywN

Posted

Great job on the challenge! You can specify images, by using <img src="" alt="" /> tag for all images and if you want to change fill property in CSS of SVG file you can use <object data="" type="image/svg+xml"></object> Sometimes using background-image is the best way, but it's almost always better to use <img> tag.

To move items relatively, for example to the left/right, top/bottom it's great to use transform property is CSS instead of margins and paddings. For example transform: translateX(-1em);. It makes code intuitive and easier to understand (also, it's easier to animate/transition transforms).

To move background-image, you can use background-position property. For example: background-position: left -10rem center; Also, if you don't want your background to repeat, you must specify background-repeat: no-repeat;.

I hope this will help you in your future projects. Have a nice day and happy coding too!

Marked as helpful

1

@snake321

Posted

Thank you so much for your valuable feedback

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord