Skip to content
Submitted 3 months ago

BMI Calculator

P
Coder-Liz•610
@Coder-Liz
A solution to the Body Mass Index calculator challenge
Preview (opens in new tab)Code (opens in new tab)
Code

Please log in to post a comment

Log in

Community feedback

  • P
    BarrieDirk•700
    @barriedirk

    It looks very nice and clean!

    Some details:

    • When it's desktop size, the logo should be 64x64.

    • You may disagree with me on this point, but I think that input labels should be "label" and not "p".

    • An improvement could be that when the input receives focus, you could make the entire text be selected with event.target.select()

    • Another improvement is to only allow numbers (and decimals if allowed) by adding your own functions for "keydown" and "input" addEventListener to prevent copying and pasting texts into the inputs.

    • Maybe the WAI-aria thing is missing

    I hope this helps.

    Best regards,

    Marked as helpful

Join 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