Assement wanted for Responsive web design and media queries

MDN Learn
#1

My Work: CodePen
Test my skills: MDN

#2

Looks like a great job to me.

#3

Great job @cyxapik_VID, congrats.

There’s just one thing I would change. you can replace:

@media screen and (min-width:40em) {
            header a {
                padding: 1.5em;
                line-height: 2em;
           }
}

To align the links.

With:

@media screen and (min-width:40em) {
            header {
                align-items: center;
           }
}

This code has the same effect as the previous one. Also I think is not necessary the border-top: 0.1em solid white because this is for mobile layout.

Try see that code: CodePen

2 Likes
#4

Thank you, Matheus, for your feedback and contribution to the community :grinning:.