Design comparison
Community feedback
- @terdoo-mzerPosted over 3 years ago
Hello there! Well done for your efforts. But a few touches here and there would be fine;
-
Maybe you can try resizing your images so that they don't occupy the whole space in the container. (I noticed this on large screens. Your images are nice on small screens however). Try using this; img { width: 100%; max-width:600px /* You can manipulate this and try to see how they respond*/ } Note: "img" is a selector. You could use whatever selector you have for you images. width: 100% ensures the image always takes 100% of whatever space allotted to it on whatever screen size. So the image scales nicely down and up when you resize the browser.
-
I do not know what screen you might be using though. But I see a horizontal scrollbar at the bottom of my screen. To deal with this, you should add this to your css; body {overflow-x:hidden}. I am sure this will deal with the overflow.
-
Your footer logo isn't visible. It is actually an svg. What I would do is, duplicate the image and rename it. Click on the svg to open in your code editor and look for "fill" property for each of the svg and change to desired color.
Happy coding!
Marked as helpful2 -
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