Latest solutions
Latest comments
- @dpiskov@Esesosa-max
Great Attempt, Approach Could be better (70 /100)
- Great Design, pretty much pixel-perfect
- Do not use a
<main>
tag to make a card because it is not semantically correct - Read HTML Semantics, it is okay to use multiple divs
- So next time, use a
<div>
and give a class of container or use another tag but do not a<main>
- Try using CSS variables it is nice!
- Do you know Flexbox? or CSS Grid?
- if not learn it, it will make centering much much easier
- Not Quite impressed, with the way you centered the
<main>
- Not Quite Responsive but the way you did it was smart
- Good Usage of Relative units like em or rem
Marked as helpful - @viniciussis@Esesosa-max
Good Attempt( 65/100), but not quite right.
- Regarding the CSS, Great Job there.
- Not quite impressed with the validation there is no custom validation
- Do not use HTML built-in validators use your own
- Yeah, the custom validation is the only thing preventing you from getting a 90/100
Marked as helpful - @Norwyx@Esesosa-max
- Flawless, Could not find any flaws (100/100)
- Good HTML Structure
- Nicely Responsive
- @francis835@Esesosa-max
- Not quite flawless but pretty great 90/100.
- The heading should not be bold.
- The HTML Structure is ok.
- @hiofadlikaakbar@Esesosa-max
Completely nothing like the original design but it is pretty cool. I just hoped you put a better anime. It is even cooler than the original design but i wouldn't employ you with that anime.
- @Nasifuad@Esesosa-max
Good Attempt, but not quite pixel-perfect because you didn't quite it right (45/100)
- Firstly, the paragraph text in the cards is just too much
- Heading text is still too much
- Padding in the card is just too much (reduce it from 30px to like 15px)
- Wrong font usage, to get the design right it is very crucial
- Please use
<p>
instead of a<h3>
in the header - Why is the paragraph text 20px? use the right font and reduce it to 17px
- Why is it NOT responsive? this made you lose points
- Try to collapse everything by giving the
.card-holder
in a media queryflex-direction:column
- Props to getting the card layout with
.middle-cards
- Try using CSS variables it is nice!
- The reason the font is not working is because you did not import the URL from google fonts, google how to do this.
Marked as helpful