darryncodes• 6,430
@darryncodes
Posted
Hi Ekrem,
Really good solution - well done!
Minor things you could consider:
- adding
cursor: pointer;
to your button styles - adding
max-width: 400px;
(px just a guide here)margin: 0 auto;
to.card
when your design changes to a column for the mobile layout so it doesn't stretch all the across the viewport - swapping
<div class="main">
<main class="main">
to clear your accessibility report. Using descriptive mark up is really important
All the best!
Marked as helpful
1
Ekrem İlkan• 90
@ekremilkan
Posted
@darryncodes Thank you very much Darryn. I will be more careful to write clean code. That was really good feedback for me.
0