Skip to content
OverviewSolutions
6
Comments
7
Ahmed Raza
@ahmedraza032

All comments

  • Ahmed Raza•60
    @ahmedraza032Posted 4 months ago

    The white background in the container of the cards is not included in the design. Rest it looks good and works fine on smaller screens

    0
  • Ahmed Raza•60
    @ahmedraza032Posted 4 months ago

    Very decent.

    1
  • Ahmed Raza•60
    @ahmedraza032Posted 4 months ago

    The solution looks close to perfect.

    0
  • Ahmed Raza•60
    @ahmedraza032Posted 4 months ago

    First of all, the shadow adds a very impressive effect to the design, so I really appreciate it.

    Secondly, the website is not completely responsive for the mobile as it should be (according to the mobile-preview image given). You can use media queries at breakpoint for mobile size (375px) and do that easily.

    Thirdly, you have written both HTML and CSS in one file, which is not a professional approach. You should take care of it in your future projects because the readability of the code is very hard in a single file.

    Fourthly, a minor detail but really important to know is the way you added border-bottom to all td tags leaves some empty space between the two columns. You can easily, avoid this issue by adding this code:

      width: 100%;
  border-collapse: collapse;
}

.row {
  border-bottom: 0.5px solid var(--stone-150);
  margin: 0px 10px;
}`

//.row is all the rows (tr) where I want border-bottom. Since I don't want it on the last row, I didn't added this class to it.
    0
  • Ahmed Raza•60
    @ahmedraza032Posted 4 months ago

    From the image it looks fine but the site is not opening.

    0
  • Ahmed Raza•60
    @ahmedraza032Posted 4 months ago

    Overall the design looks fine but the specific design properties for hover state is missing like when we hover over the link, it should change it's color.

    1
  • Ahmed Raza•60
    @ahmedraza032Posted 4 months ago

    I don't see any missing details in the solution.

    0