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

Sunnyside Agency Desktop

@juliajgp

Desktop design screenshot for the Agency landing page coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hi! This is my first challenge here, and I'm a newbie. I'd love some feedback to improve and keep working.

Community feedback

amichno 220

@amichno

Posted

Hi Julia, congrats! Add underline to the "learn more". Website has it but only on the active statement. You can do it by adding an after pseudo-elementto the text.

Greetings, Anna

Marked as helpful

1

@jglopezre

Posted

Congrats.. I suggest to you, at footer, in social links, you have to wrap every <img> into an anchor <a>, for give to icons the possibility to send user to respective page, and you could give to <a> margin-left and margin.right to open space between. Optionally, you could give to these new <a> option to hoverize (a:hover) giving interactive response when user pass mouse pointer over a icon.

Greetings.

Marked as helpful

1
Dev Rathore 2,600

@GitHub-dev12345

Posted

Congratulation to complete the challenge 🚀 My small Suggestion for your project, In Paragraph Section you gave the background color; the color given in style guide, go check and apply the color: 😊

And in Social Media Icons container give the margin & padding - Your social media looks very osm 👍

I hope you find this helpful

Marked as helpful

1

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