Need assessment for Fundamental Layout Comprehension

MDN Learn
#1

Peace to all.

Here is the task

Here is the solution

Thank you all for your responsiveness!

#2

Hello @rab_Allaha

Great job! :medal_sports: Nice addition of the media query for smaller screens.

Just two small notes:

  • You could remove flex-direction: row; and position: sticky; from nav ul . The first is the default behavior and the second is already set on nav itself.
  • In .feature you should change padding to margin. It doesn’t make any difference in this task, but we want the outer distance to be 20px. Try adding border: 2px solid black; to .feature to see the difference.

Have a nice day,
Michael

2 Likes
#3

Hello @rab_Allaha and @mikoMK

plus what @mikoMK said if you resize the window you would notice that the text of the menu (specialy the one with 2 words) does not align correctly you can fix that by

a{
display: inline-block;
text-align: center;
}

hope that help and have a nice day both of you :slight_smile:

2 Likes
#4

NIce catch, @justsomeone :heart:

2 Likes
#5

thanks boss @mikoMK :joy:

2 Likes
#6

Hello @mikoMK

Fixed the errors you pointed out, can you check the task now?

1 Like
#7

Assalamu alaikum. Also made a correction :slight_smile:

2 Likes
#8

Great correction! Now it’s perfect :+1:

1 Like
#9

well done @rab_Allaha

1 Like