P
Matt Studdert• 13,611
@mattstuddert
Posted
Nice work Muhammad! The main pointers I've got are:
- You're currently only setting the mobile background image. So I'd add the desktop version in for your desktop layout and also set
background-size: cover;so that it stretches the width of the screen. - Your "Register" anchor tag doesn't need
role="button", as it would just be a regular clickable link if this were a fully-functioning website.
Keep up the great work!
0
@fadzrinmadu
Posted
@mattstuddert hmm i forget some details. thanks for the comments i will fix it
0
