@Arkadiusz-coder
Posted
Hi,
your solution looks nice and works all just fine but design file provided by Frontend Mentor sagest that after a number is selected by user the circular area of the given number (your span element) should stay gray, unless user will change hers mind I choose another number. I will not give you my solution to this since it is good practice to figure it out by yourself, and I saw you are not far from this in your project, cause the color of the given span is changing, but only for a moment.
I've noticed that in your HTML file you left section provided by original file. I guess it would be a good practice to remove it to your style.css file, just for better order of the project.
And last one thing: you insert images in HTML by using <svg> and <path> elements, within which you have a lot of numbers. Are they necessary? In my solution I just used something like this: <img src="./icon-star.svg" alt="icon-star" class="star-image"> And that's works pretty well.
I hope this will be helpful. I would be glad if you have any comment on my projects here :)
Marked as helpful
