Vanza Setiaβ’ 27,795
@vanzasetia
Posted
Hi, Elysson Tanaka! π
Congratulations on finishing this challenge! π
I have a few suggestions for this solution.
- For the logo, I would recommend improving the alternative text by telling the users the company name which is "Ping". This will help users know that they are on the Ping site or the site is called Ping.
figure
element doesn't need to wrap everyimg
element. The only reason to usefigure
is if you need to include afigcaption
. Otherwiseimg
tag is fine.- I would recommend moving the
button
inside theform
element. Also, add thetype="submit"
to thebutton
element. - Avoid using JavaScript to add styling (unless you've no other option). JavaScript allows you to change the CSS code using the
style
property. But, to make debugging easier and improve code maintainability, itβs best to avoid it. Use CSS classes instead.
I hope this helps! π
Marked as helpful
0
Elysson Tanakaβ’ 90
@elyssontanaka
Posted
@vanzasetia thank you so much for your feedback! I'll take those suggestions in consideration in future projects! I'll fix my code as well π
0
Vanza Setiaβ’ 27,795
@vanzasetia
Posted
@elyssontanaka No problem! π
0