@thomashertog
Posted
you may have completed the desktop design, but it is not responsive. I can't see the mobile view of this.
other than that, you have used absolute positioning for everything (with pixels) that is definitely not how layouting works nowadays. maybe you can try learning some things about flexbox and/or CSS grid to make a new attempt at this challenge.
your solution is tagged with #accessibility yet you still have <div class="button">
with an svg (for the icon) which is not hidden from assistive tech (as it should be since it's only decorative), and a paragraph for the text, but no <button>
element is used...
Marked as helpful
@eslamwaleed1
Posted
Thanks a lot bro! Your feedback really helps. @thomashertog
@thomashertog
Posted
@eslamwaleed1 don't be hesitant to ask for help in #help on Discord if you have trouble implementing the feedback
Marked as helpful
@eslamwaleed1
Posted
Thanks again (": @thomashertog