-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Develop #4074
base: master
Are you sure you want to change the base?
Develop #4074
Conversation
This reverts commit 9affec8.
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 code is great, there's just one organization tip that should stick more to you. Good job!
The only problem I found is that you made a mistake in the link 😅 I'll just ask you to fix it so I can approve your PR
src/style.css
Outdated
margin: 0; | ||
} | ||
|
||
.stars--1 .stars__star:nth-child(-n+1), .stars--2 .stars__star:nth-child(-n+2), .stars--3 .stars__star:nth-child(-n+3), .stars--4 .stars__star:nth-child(-n+4), .stars--5 .stars__star:nth-child(-n+5) { |
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.
You should always break lines when there's multiple selectors for the same properties
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.
Hey @jv-aquino. I fixed the problems. Please check if everything is ok.
- [DEMO LINK](https://<your_account>.github.io/layout_stars/) | ||
- [TEST REPORT LINK](https://<your_account>.github.io/layout_stars/report/html_report/) | ||
- [DEMO LINK](https://mfcastilho>.github.io/layout_stars/) | ||
- [TEST REPORT LINK](https://mfcastilho>.github.io/layout_stars/report/html_report/) |
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.
Here you just forgot to remove the > 😅
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.
Good job! The link is still wrong in the README but this doesn't affect the solution, just remember to update it so people can see your site without problems
DEMO LINK
TEST REPORT LINK
Yellow stars are added with container modifier + pseudo-selector (NO extra classes)
Each BEM block has its own separate file
All
Typical Mistakes
fromBEM
lesson theory are checked.Code follows all the Code Style Rules ❗️