I see that, you have cleared the test, but if your layout doesn’t look good, and if you pass the test, then it’s like only 50% you have cleared.
Your navbar should hold a structure as ul > li > a, then
<ul> should be the parent of
<li> elements, it is a list item, so it tends to stay within the
<ol>, so change your structure accordingly.
<ul> has a default left-padding and margin at top and bottom, you might want to clear it for some cases or have a knowledge of it.
To have a border for
li elements, you might give the border property to the
li, rather than the anchor elements, but do this after you alter the structure. After you change the structure of your navbar, you can see the border you have given to
ul. Currently it overlaps the last anchor element, because of child mismatch