Skip to content

Conversation

dinotudor
Copy link

No description provided.

max-width: 100%;
}

@media (max-width: 768px) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

You don't need to put all your code inside this @Mediaquery. Just specific things that only are different for that screen size. Please if you have questions regarding how to use media queries let us know.

.ul-parent {
/* margin-right: 10px; */
display: flex;
flex-direction: row;
Copy link
Collaborator

@tawebbcn tawebbcn Apr 8, 2019

Choose a reason for hiding this comment

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

the default value for flex-direction is already row.

<meta http-equiv="X-UA-Compatible" content="ie=edge">
<script src="https://use.fontawesome.com/1fb0eb3889.js"></script>
<link href="https://fonts.googleapis.com/css?family=Heebo:400,700" rel="stylesheet">
<!-- <link rel="stylesheet" type="text/css" href="./stylesheets/reset.css"> -->
Copy link
Collaborator

Choose a reason for hiding this comment

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

you commented out the reset file so the reset isn't taking place.

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, though you reused a lot of properties in different selectors, the css could be refactored to be less repetitive.

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