Design comparison
Community feedback
- @denieldenPosted about 2 years ago
Hi Krishna, congratulations on completing the challenge, great job! ๐
Some little tips for optimizing your code:
- To fix the top image in the background just put more specific background properties to the body:
background: url("../img/pattern-background-desktop.svg") no-repeat top center; background-size: contain; background-color: #e0e8ff;
- remove all unnecessary code, the less you write the better as well as being clearer: for example the
div
container of image - use
min-height: 100vh
to body instead ofheight
, otherwise the content is cut off when the browser height is less than the content - instead of using
px
use relative units of measurement likerem
-> read here
Hope this help! Happy coding ๐
Marked as helpful1@krishna-cyberPosted about 2 years ago@denielden Hello deniel,๐Iam so happy to you, I have done some changes on my code ๐ป, according to your suggestions. It seems very helpful ๐ and clearcut code and pattern ... can you check once my code again and kindly please give some tips for optimizing my code๐๐ I am learning now. Some of the changes i have done in my code: ๐Remove an unnecessary div that wraps img tag ๐Remove an unnecessary div at pricing section ๐Background style changed as you mentioned me above
and IT WORKS FINE NOW.
1@denieldenPosted about 2 years ago@krishna-cyber Sure, now is much better !! You are welmone and keep it up :)
Marked as helpful1@krishna-cyberPosted about 2 years ago@denielden Thanks a lot .... very helpfun and kind person ๐
1@denieldenPosted about 2 years ago@krishna-cyber Thanks you too :)
Marked as helpful1@krishna-cyberPosted about 2 years ago@denielden have you watched my another solution tooo ..."product review card".... waiting for you feedback
0
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