
Solution retrospective
It took me about half an hour to do it, I need your opinion on the design and the code . If it is a simple modification, it will help me a lot, thank you in advance.
Please log in to post a comment
Log in with GitHubCommunity feedback
- @WebDevCampos
Hi there! You did well, but if I may, there are a few corrections to make. Firstly, the background-image is repeating, you can fix that setting
css background-repeat:no-repeat
. Secondly, please review the component's width and height. You don't have to set a relative unit such as percentage, you can set a fixed size. Read thestyle-guide.md
file and you'll see all the requisites for the project. The size for mobile devices for instance, it's375px
and for desktops it's1440px
, maximum. All projects must have anh1
tag, so you can replace theh3
and set thefont-size
till it gets close to final result. You can check thestyle-guide.md
file to set the proper colors to the elements as well. Any questions or further assist, don't hesitate to reach me out! Kind regards! - @Wr3nD
you should always follow correct colors ( you propably missed it, font color is wrong ) besides that, there is not much to say about that project bcouse of it simplicity
Join 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