add task solution #5190
add task solution #5190
Conversation
There was a problem hiding this comment.
Great job on implementing the task requirements! 🎉 You've structured your code well and met all the core requirements. There are just a couple of minor issues to address in the future: correct the typo in the class name &__discription to &__description, and ensure the font-weight property doesn't have a unit. Keep up the good work! 🚀
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| width: 100%; | ||
| justify-content: space-between; | ||
|
|
||
| &__discription { |
There was a problem hiding this comment.
There is a typo in the class name &__discription. It should be &__description to match the HTML class name and maintain consistency.
| height: 40px; | ||
| border-radius: 5px; | ||
| background-color: $accent-color; | ||
| font-weight: 700px; |
There was a problem hiding this comment.
The font-weight property should not have a unit. It should be font-weight: 700; instead of font-weight: 700px;.
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 ❗️