Happy Phiri
@happy-phiriAll comments
- @priyanshu1443Submitted over 1 year ago@happy-phiriPosted over 1 year ago
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 - @EstherNyumuSubmitted over 1 year ago
Can anyone assist me on the best way to align the divisions so that they don't collide when I resize my browser?
@happy-phiriPosted over 1 year agoGood 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 helpful1 - @CBZ86Submitted over 1 year ago@happy-phiriPosted over 1 year ago
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 helpful0 - @aitoriasSubmitted over 1 year ago
- @EYRAM565Submitted almost 2 years ago
feedback on how to improve on the layout especially its responsiveness.
@happy-phiriPosted almost 2 years agoGood 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 helpful0