Skip to content

Conversation

Gusbe
Copy link

@Gusbe Gusbe commented Apr 4, 2019

No description provided.

Copy link
Collaborator

@tawebbcn tawebbcn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job!

nav grey: #5b5e6d;
*/

/* ---- reset ---- */
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reset at the top, good

flex-direction: column;
}

nav {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This si not reset anymore, maybe you could add a breaking comment to specify that


nav {
display: flex;
flex-direction: row;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

By default flex-direction is row

#menu {
margin: 10px;
}
.bar-menu {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Try to be consistent with the format. If you put a blank space between the rules, put it all the time


#links ul {
display: flex;
flex-direction: row;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants