Skip to content
OverviewSolutions
1
Comments
8
Sahil Yadav
@Sahil-Yadav

All comments

  • Sahil Yadav•70
    @Sahil-YadavPosted over 3 years ago

    Do you use a bigger display screen or it's just not centered?

    1
  • Sahil Yadav•70
    @Sahil-YadavPosted over 3 years ago

    Everything is good but in the desktop view the whole thing is shifted to the top(it should be in the middle) and the "per month" should be also aligned to the "$29".

    0
  • Sahil Yadav•70
    @Sahil-YadavPosted over 3 years ago

    When it is in mobile view there is no space on the top and the bottom of the whole thing. There is unnecessary space below the discover paragraph(in the desktop view). Everything else is good.

    1
  • Sahil Yadav•70
    @Sahil-YadavPosted over 3 years ago

    If you see carefully there are shadows given to the divs(but this is not a problem). But all the content in shifted to the top, it isn't in the middle. Everything else is good.

    1
  • Sahil Yadav•70
    @Sahil-YadavPosted over 3 years ago

    You haven't added shadows to the divs. And there is unnecessary white space at the bottom of kira's div.Everything else is good.

    0
  • Sahil Yadav•70
    @Sahil-YadavPosted over 3 years ago

    Your top headers should be aligned left. Everything else is good.

    0
  • Sahil Yadav•70
    @Sahil-YadavPosted over 3 years ago

    One thing you can do is giving the image that color background and then blend it with the background Or you can implement the image through css background-image property and then use linear-gradient.

    1
  • Sahil Yadav•70
    @Sahil-YadavPosted over 3 years ago

    It's good but there is a lot of unnecessary white space that shouldn't be there. That is easy to fix.

    0