@SJ-Nosrat
Posted
Hi Dawid, Great solution, it looks identical to the design. You have a good eye for the details.
Just a few comments, with respect to the accessibility in your HTML, try and use semantic tags. For example: instead of using: <div class="main-wrapper"> do the following <main class="main-wrapper"> that way you provide better experience for screen readers.
Also good use of the CSS GRID; I should start using it for my layouts. Other than that maybe instead of using pixel, try and use rem and percentages for your widths; allows for more fluid responsiveness.
Keep up the great work!
I Would appreciate if you could review my attempt at this solution too; would love your feedback.
Thanks in advance!
Marked as helpful
@dadko44
Posted
@shahin1987
Hello, thank You for your feedback.
Tomorrow I'll come back an try to encompass your tips into my solution. I'll be more than happy to take a look at your solution too, hope You're having a good day / evening