Skip to content
OverviewSolutions
3
Comments
5
Haythem Lazaar
@HaythemLazaar

All comments

  • SahilShameerDev•40
    @SahilShameerDevSubmitted over 3 years ago

    The Transition Between Themes Is Not So Smooth. So any tip on that will be so helpful. Thank You

    Html,Css,Js Website

    2
    Haythem Lazaar•60
    @HaythemLazaarPosted over 3 years ago

    Maybe add a transition delay between color modes but it looks fine.

    0
  • Haythem Lazaar•60
    @HaythemLazaarPosted over 3 years ago

    Looks nice John, nothing feels wrong. you did a very good job, the only thing I would've added is some kind of interaction with the cards maybe when hovered it scales a bit.

    1
  • Haythem Lazaar•60
    @HaythemLazaarPosted over 3 years ago

    Using display: flex; you just need to set flex-direction: column-reverse; and by that I think the image might pop up top, Hope this helps.

    0
  • Haythem Lazaar•60
    @HaythemLazaarPosted over 3 years ago

    Hey, I think it's better if you center the card div using display: grid; and place-items: center; other than that it looks good maybe just size the card down a bit.

    1
  • Haythem Lazaar•60
    @HaythemLazaarPosted over 3 years ago

    Hey you might wanna change the padding-top on .infobox, also I think the mobile version isn't done yet, you might consider starting mobile first from now on it eases things up really, well it did for me.

    0