Thomas Hertog• 715
@thomashertog
Posted
the solution is almost looking like the design, yet there are some serious accessibility improvements to be made.
- you still have a lot of
pxvalues forwidth(and evenheight, which would be much better asmin-height) so it isn't responsive in any way - there is no semantic HTML being used (e.g.
<main>) - the rating is chosen from
<li>instead of using radio-buttons in a<form> - heading levels are not being used consistently (imagine them like the input for a table of contents)
Marked as helpful
1
Hüsrev• 60
@hsrvms
Posted
@thomashertog Thanks for your valuable comments, I will take them into consideration on my future projects.
0
