Skip to content
OverviewSolutions
15
Comments
5
Happy Phiri
@happy-phiri

All comments

  • Happy Phiri•410
    @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
  • Esther Nyumu•50
    @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?

    Grid component using CSS

    1
    Happy Phiri•410
    @happy-phiriPosted over 1 year ago

    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
  • Happy Phiri•410
    @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 helpful
    0
  • Happy Phiri•410
    @happy-phiriPosted over 1 year ago

    Good job!!!

    1
  • Happy Phiri•410
    @happy-phiriPosted almost 2 years ago

    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