Javier Duarte• 210
@javiIT10
Submitted
Ahmed Raza
@ahmedraza032
All comments
- Ahmed Raza• 60
@ahmedraza032
Posted
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 - Davi Martins Freire• 160
@davimartins017
Submitted
Ahmed Raza• 60@ahmedraza032
Posted
Very decent.
1 - Edu209C• 80
@Edu209C
Submitted
Ahmed Raza• 60@ahmedraza032
Posted
The solution looks close to perfect.
0 - pamithapandian• 50
@pamithapandian
Submitted
Ahmed Raza• 60@ahmedraza032
Posted
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 - Davi Martins Freire• 160
@davimartins017
Submitted
Ahmed Raza• 60@ahmedraza032
Posted
From the image it looks fine but the site is not opening.
0 - Danae Lescano• 270
@Lescano713
Submitted
Ahmed Raza• 60@ahmedraza032
Posted
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 - Alexandru Stroe• 100
@alstrow
Submitted
Ahmed Raza• 60@ahmedraza032
Posted
I don't see any missing details in the solution.
0
