Design comparison
Solution retrospective
I hope it's good enough for qualifying 😄 Give me your feedback !!
Community feedback
- @brodiewebdtPosted almost 3 years ago
@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.
2 - @Nam-HaiPosted almost 3 years ago
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 paswordtype="password"
1@smalanceaPosted almost 3 years ago@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'
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
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