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

@MAR2409


Design comparison


SolutionDesign

Solution retrospective


This was a fun challenge to complete. However, I was unable to make the the logos change color when hovering. If any feedback on how to accomplish that will be greatly appreciated so I can then update the code. Thank you all so much.

Community feedback

Hassia Issah 50,670

@Hassiai

Posted

Replace the section tag with the main tag to fix the accessibility issues. there is no ne for <a> within the <button> remove it to fix the error issues. in the css give the button a cursor value of pointer.

for the right and left-sides of the image give the nav, the main and the footer a max-width of 1440px, a width of 80% and margin: 0 auto.

nav, footer, main{
max-wisth:1440px;
width: 80%;
margin: 0 auto;
}

change the value of background size in the mobile design to contain and in the desktop design to cover.

in the desktop design give .landing_img a width of 55% and .landing_text a width of 40%. and no need for margin-right value in .landing-text.

Hope am helpful.

Well done for completing this challenge. HAPPY CODING

Marked as helpful

1
Redha 90

@med-redha

Posted

i think if u add &.logos class name : hover {box-shadow } and a transition to the logos class name would do the trick .

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