Develop #5157
Develop #5157
Conversation
There was a problem hiding this comment.
Great job on your implementation! 🎉 The code meets most of the task requirements, and the structure is well-organized. Just a small note: ensure that the stars__starN class name is consistent with your naming conventions unless it's intentionally different. Keep up the good work and happy coding! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| <div class="stars__star"></div> | ||
| <div class="stars__star"></div> | ||
| <div class="stars__star"></div> | ||
| <div class="stars__starN"></div> |
There was a problem hiding this comment.
The class name stars__starN seems inconsistent with the other class names like stars__star. If this is not intentional, consider renaming it to maintain consistency.
| background-repeat: no-repeat; | ||
| } | ||
|
|
||
| &__starN { |
There was a problem hiding this comment.
The class name stars__starN might be inconsistent with the naming convention used for other elements like stars__star. If this is not intentional, consider renaming it to maintain consistency.
DEMO LINK
TEST REPORT LINK
there 2 BEM blocks
cardandstarseach in their own fileSCSS Nesting is used for
elements,modifiersandpseudo-classesSCSS Variables are used for main values and placed in a separate file
all
stars--Nmodifiers work as expected (Highlight firstNstars)Code follows all the Code Style Rules ❗️