Submitted over 3 years ago
HTML, CSS and JavaScript were the technologies used.
@lawrecks
Design comparison
SolutionDesign
Solution retrospective
What do you think about this? Where do you think I need to improve on?
Community feedback
- @yasssuzPosted over 3 years ago
Hello Ugo!
Awesome build, just a few tips:
- Your background is repeating on bigger screens, this is happening because the image is not of the same size. A simple solution would be to add
background-repeat: no-repeat; background-size: cover;
. This will stop your background from repeating and will make sure that your background is the same size as your screen without stretching it, it's contained automatically. - I strongly suggest using unordered lists for semantic reasons. Use
ul
onid="todoList"
and for all the children useli
. - U should not use a percentage width on your
class="to-do"
, this is making your main content look very small on smaller screens. I suggest using550px
instead of35%
.
Please upvote my comment if I was helpful and happy coding :)
Marked as helpful1@lawrecksPosted over 3 years ago@Galielo-App Thanks for your feedback. I'll implement the changes as soon as possible.
0 - Your background is repeating on bigger screens, this is happening because the image is not of the same size. A simple solution would be to add
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord