Matt Seidel• 570
@mseidel819
Posted
I like the creativity on the button colors!
a couple accessibility fixes:
- Change
<div class="continer">to<main class="continer"> - Change
<div class="attribution" role="footnote">to <footer class="attribution"> - I think you can also get away with changing
<h2 class="heading">How did we do?</h2>to an<h1>element. - on line 12 of your index.html, in
type="text/javascript".
Other things you can look into:
- when you hover over the number buttons and the submit, cursor should be a pointer.
cursor:pointer. - when you hover over submit, the background should change to white, and the text should change to orange.
Marked as helpful
1
BiscuitCandy• 100
@BiscuitCandy
Posted
@mseidel819 Thank you for the feedback! I always want to try making my html a bit professional but not know when to use those tags. Your fixes gave me the insight I needed.
0
