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

Responsive Loopstudio landing page

InterceptorTTā€¢ 390

@InterceptorTT

Desktop design screenshot for the Loopstudios landing page coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


Cool challenge! I struggle a little bit in figurin' out how to do the second section (interactive), and as you can see the part of the text is not properly aligned with the rest of the layout. I'm curious to see how you did it! any suggestion would be appreciated anyway!

Community feedback

@VCarames

Posted

Hey there! šŸ‘‹ Unfortunalty, there are a lot issues that I would not expect at this challenge level... I would highly recommend starting with an easier challenge.

Here are some suggestions to help improve your code:

  • Your navigation needs to be rewrittenāš ļø. There should not be two separate codes for your navigation menu, there should only be one and you will use CSS to style it for mobile/desktop.
  • Since your logo is wrapped inside an anchor element, you need to include āš ļø an aria-label stating the logo's name ("Loopstudios") and where it leads to ("Home").
  • The nav toggle should be the first thing inside of the nav to improve accessibility and it shouldāš ļø be a button not an anchor. It should also have aria-expanded, area-label and an aria-control.
  • Remove all the br elements you have added, they are being used incorrectlyšŸš©.
  • For the card, instead of doing this [example provided at bottom of text], it far better to use the picture element.

āŒ:

 <a href="#" class="img-desktop"><img src="images/desktop/image-deep-earth.jpg" alt="#"></a>
<a href="#" class="img-mobile"><img src="images/mobile/image-deep-earth.jpg" alt="#"></a>
  • In the card, the images should notāŒ be the element that is interactive, it should be the text and you have the text "breakout" to make the entire card interactive.

Here is a helpful article on how to achieve that https://piccalil.li/tutorial/create-a-semantic-break-out-button-to-make-an-entire-element-clickable/

If you have any questions or need further clarification, feel free to reach out to me.

Happy Coding! šŸ‘¾

Marked as helpful

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