@ahmedibrahim108Submitted 2 months ago
Miguel Nobre
@MiggySmallzAll comments
- @MiggySmallzPosted about 2 months ago
Looks Good! Maybe just round the corners of the card to keep the same visual cues. Good job at keeping the colors too.
Marked as helpful0 - @bezocSubmitted 2 months ago@MiggySmallzPosted about 2 months ago
Looks pretty good! A few tips:
The "attribution" div could be put under the card instead of beside it. You can do this by having a "container" div that encompasses both the card and the "attribution" div, set the container's display: flex and then set the flex-direction: column.
Not sure if you were using the Figma file given, but the exact color of the background is given there.
Overall looks great!
0 - @yours-truly68Submitted 2 months ago@MiggySmallzPosted 2 months ago
Nice job! Looks great! Just want to point out that the margins are a little too small compared to the original as well as the font sizes are a bit too big.
0