Skip to content
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 #3499

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

reznik-denis
Copy link

@reznik-denis
Copy link
Author

Не розумію чому не проходять тести, потрібна допомога, я так зрозумів в завданні хочуть щоб клас модифікатор стояв на блоці один, і в залежності від числа в ньому кількість елементів замінялась на активні зірки, я знаю як це зробити через js але як це зробити лише за допомогою css не можу зрозуміти, перепробував різні варіанти, почитав гугл, подивився уроки і ніде нічого не знайшов, як взяти число із батьківського класу і засунути його в формулу для nth-child не розумію.

src/index.html Outdated
<div class="stars__star"></div>
</div>
<div class="stars">
<div class="stars__star stars--1"></div>

Choose a reason for hiding this comment

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

do not add modifier for star's what need to be active, add it for parent element and control the active stars through scss

Copy link
Author

Choose a reason for hiding this comment

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

done

src/style.css Outdated
Comment on lines 25 to 29
.stars--1,
.stars--2,
.stars--3,
.stars--4,
.stars--5 {

Choose a reason for hiding this comment

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

you should use nth-child pseudo-class here to control active stars

Copy link
Author

Choose a reason for hiding this comment

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

Зробив, та мені наче попередній варіант був кращий, сподіваюсь що саме цього і хотіли, бо тести пройшли

src/index.html Outdated
@@ -1,3 +1,4 @@
<!-- #region head -->

Choose a reason for hiding this comment

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

remove comment

Copy link
Author

Choose a reason for hiding this comment

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

done

src/index.html Outdated
@@ -7,7 +8,49 @@
<title>Stars</title>
<link rel="stylesheet" href="./style.css">
</head>
<!-- #endregion -->

Choose a reason for hiding this comment

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

remove comment

Copy link
Author

Choose a reason for hiding this comment

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

done

@andriy-shymkiv
Copy link

Не розумію чому не проходять тести, потрібна допомога, я так зрозумів в завданні хочуть щоб клас модифікатор стояв на блоці один, і в залежності від числа в ньому кількість елементів замінялась на активні зірки, я знаю як це зробити через js але як це зробити лише за допомогою css не можу зрозуміти, перепробував різні варіанти, почитав гугл, подивився уроки і ніде нічого не знайшов, як взяти число із батьківського класу і засунути його в формулу для nth-child не розумію.

answered for this question in comments, please read them

<div class="stars__star"></div>
<div class="stars__star"></div>
<div class="stars__star"></div>
</div>
Copy link

Choose a reason for hiding this comment

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

Add an empty line between each block

Copy link
Author

Choose a reason for hiding this comment

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

done

src/style.css Outdated
Comment on lines 25 to 43
.stars--1 > .stars__star:nth-child(-n + 1) {
background-image: url(./images/star-active.svg);
}

.stars--2 > .stars__star:nth-child(-n + 2) {
background-image: url(./images/star-active.svg);
}

.stars--3 > .stars__star:nth-child(-n + 3) {
background-image: url(./images/star-active.svg);
}

.stars--4 > .stars__star:nth-child(-n + 4) {
background-image: url(./images/star-active.svg);
}

.stars--5 > .stars__star:nth-child(-n + 5) {
background-image: url(./images/star-active.svg);
}
Copy link

Choose a reason for hiding this comment

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

Combine them to avoid code duplication

Copy link
Author

Choose a reason for hiding this comment

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

done

Copy link

@andriy-shymkiv andriy-shymkiv 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!

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.

3 participants