Priyanshu farkare• 270
@priyanshu1443
Submitted
Happy Phiri
@happy-phiri
All comments
- Happy Phiri• 370
@happy-phiri
Posted
Great work on this. One thing I noticed is that you must have forgotten to add the close button (X) for the mobile nav. So currently the nav on mobile display does not close once opened.
0 - Esther Nyumu• 50
@EstherNyumu
Submitted
Can anyone assist me on the best way to align the divisions so that they don't collide when I resize my browser?
Happy Phiri• 370@happy-phiri
Posted
Good job on the project. What you could do to improve layout solve the problem of text overflowing or been hidden is removing the fixed height 32vh on the community benefits div and introductory part div. Height should be removed altogether or set to auto.
Marked as helpful
1 - Nicolas Lugo• 150
@CBZ86
Submitted
Happy Phiri• 370@happy-phiri
Posted
Good job on the project. One improvement you could make is working on the responsiveness of the site. You can review screen breakpoints you've used and also review when it could be best to used grid or flexbox. Otherwise, good job.
Marked as helpful
0 - Aitor de Diego• 40
@aitorias
Submitted
Happy Phiri• 370@happy-phiri
Posted
Good job!!!
1 - Eyram• 270
@EYRAM565
Submitted
feedback on how to improve on the layout especially its responsiveness.
Happy Phiri• 370@happy-phiri
Posted
Good solution here. One thing that could improve the layout could be adding a container class that you'll use to add some padding to the body. Another could be adding the hover state to the actual heading (h2, h3) rather than to the whole div, like so - .container1 h2:hover {}. This should add the hover states to the h2 in container1 div rather than to the whole div. Otherwise well done.
Marked as helpful
0
