@Code-Beaker
Posted
Hi there, congratulations on completing this project. You've done a great job with this one!
I have some small suggestions regarding your website that might help you make it better:
- Use
class
instead ofid
in your HTML. - I attempted to visit your website on a desktop and I noticed that, the
<img id="img2" src="./images/illustration-2.svg" alt="" width="500px"/>
was overflowing my screen (image from "Stay productive, wherever you are" section).
This can be prevented by removing the width="500px"
from your html and adding reset to your css:
img {
max-width: 100%;
}
- instead of flex, for a better and easier layout control, you can your
display: grid
for your sections. Like this:
#content2 {
display: grid;
grid-template-columns: repeat(2, 1fr);
}
this will be better for handing 2-column layout compared to flex
.
Note:
change the id
selector to either a class
or to your liking...
-
The footer links have poor contrast making it hard to see and read. Try a lighter color for them.
-
Mobile responsiveness of the site still needs some work. Currently, it looks like the width of the whole body is shrinking and not the elements.
I hope you find this helpful... 😊
@SuriyanRavi
Posted
@Code-Beaker Thank you for your suggestion. I Seriously dont know what the error in responsive width. I tired much to solve it but i couldn't. If you know help guide and help me with responsiveness.
@Code-Beaker
Posted
Hi, @SuriyanRavi
I'm sorry about that, it looks like the responsiveness is now fixed or it was an error from my side. You've done well with the responsiveness and now it looks good.
But there are still some things that need some work that I have mentioned in my original comment.
Good luck! 👍
Marked as helpful