Looking to hire developers?
I hope it's good enough for qualifying 😄 Give me your feedback !!
@brodiewebdt
Posted
@Nam-Hai is right you did a great job with this.
Wrap your container div in a Main tag and it will clear the accessibility warnings.
Download AXE DevTools and you can clear accessibility warnings while you code. https://www.deque.com/axe/devtools/
Hope this helps.
@smalancea
Posted
@brodiewebdt Done, thank you !!
@Nam-Hai
Posted
You nailed the design dude ! nice work
Using margin: 0 auto to center the main container is a good habit, instead making body a flexbox.
What could have been improved :
Give your input a type, for email type="email", for pasword type="password"
@smalancea
Posted
@Nam-Hai I actually used the right type for email and password, i see clearly that for email is type = 'email', and for password is type = 'password'
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord
