P
Srijan Manandhar• 450
@srijanss
Posted
- Solution includes semantic HTML, however it would be meaningful to add main heading and form component in main element instead of header landmark
- solution is accessible using keyboard, but it moves focus to result element and some other element on the page as well which can be bit confusing
- Layout looks good on all the screen sizes
- code is well-structured, readable and reusable
- solution differs a bit from design, some styles for headings are different, and on page load welcome message should be shown instead of the bmi result information
- really nice solution , keep up the good work
0
