-
Notifications
You must be signed in to change notification settings - Fork 5.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add task solution #4683
add task solution #4683
Conversation
ser1stydent
commented
Mar 5, 2024
•
edited
Loading
edited
- DEMO LINK
- TEST REPORT LINK
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<a class="nav__link laptops" data-qa="hover" | ||
href="#laptops s computers">Laptops & Computers</a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<a class="nav__link laptops" data-qa="hover" | |
href="#laptops s computers">Laptops & Computers</a> | |
<a | |
class="nav__link laptops" | |
data-qa="hover" | |
href="#laptops s computers" | |
> | |
Laptops & Computers | |
</a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
padding: 0 50px; | ||
display: flex; | ||
justify-content: space-between; | ||
left: 0; | ||
right: 0; | ||
align-items: center; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rewrite styles in correct orders https://9elements.com/css-rule-order/
Fix all cases
align-items: center; | ||
} | ||
|
||
.logo { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix all cases (remove redundant whitespaces)
.logo { | |
.logo { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please, fix all previous comments and fix your demo inks
if you have some problems with that then feel free to ask for some help in the chat
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Your demo now works but the issues are still the same, fix it and don`t forget to commit them and push