-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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 #5437
base: master
Are you sure you want to change the base?
add task solution #5437
Conversation
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.
Thank you for your hard work on this project! 🌟 Your implementation is mostly on point, but there's a small issue that needs to be addressed before we can approve the pull request. The extra <!doctype html>
declaration at the end of your HTML file should be removed to prevent any potential parsing issues. Once that's fixed, your code will be ready to go! Keep up the great work, and I'm looking forward to seeing the updated version. 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
🔄 Fix the issues, commit the changes, and then re-request my review using the Re-request review
button in the Reviewers tab.
src/index.html
Outdated
</body> | ||
</html> | ||
<!doctype html> |
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.
There is an extra <!doctype html>
declaration at the end of the file. This is unnecessary and should be removed to avoid potential issues with HTML parsing.
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.
Review Summary
Great job on your submission! 🎉 Your code meets the task requirements and constraints, and you've effectively addressed the previous issue regarding the extra <!doctype html>
declaration. Here are some key points from the review:
-
HTML Structure: Your HTML is well-structured, following the requirement to have 6
stars
blocks with 5stars__star
elements each. The appropriate modifiers are applied correctly. -
CSS Styling: The CSS is well-organized, effectively styling the elements as intended. The use of the
nth-child
selector is spot-on for targeting specific stars. Thebackground-image
for stars is correctly used, and you have adhered to the requirement of not using thegap
property. -
Formatting: Both your CSS and HTML formatting generally adhere to style guidelines, such as using
display: flex
and resetting the browser's default margin.
While your code is approved, remember to always review feedback to improve your skills. Addressing issues, even minor ones, is crucial for your growth as a developer. Keep up the great work, and continue learning from each experience! 🚀
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
DEMO LINK
TEST REPORT LINK