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

Landing page: Using CSS & HTML.

@tafnesj


Design comparison


SolutionDesign

Solution retrospective


I'd like to receive feedbacks from my code.

Community feedback

@justAdevTV

Posted

Looks pretty good on my desktop and phone!

Suggestions for improvement:

  1. At 874 width, the site starts to look a little tight. It wouldn't hurt to add another breakpoint here.
  2. For Buttons, I'd suggest using padding instead of setting the width. Check this out: https://www.w3schools.com/css/tryit.asp?filename=trycss_buttons_basic
  3. My mouse doesn't turn into a "pointer" when I hover over the social buttons. You should add an <a> tag for accessibility too.

Overall, nice work!

Keep it up 🙌

3

@tafnesj

Posted

@justAdevTV Than you for the feedback! I will consider it to improve it.

6

@justAdevTV

Posted

@tafnesj Awesome, let me know if you update it :)

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