Age-calculator-app Solution

Community feedback

• 

  P

  Giorgi Khaburzania•350

  @GioKhabuPosted 9 months ago

  Hello Eftimie,

  Nice work, functionality works perfectly. I would just suggest few points:

  1. First of all for inputs you can add placeholders like it is in the initial design DD, MM, YYYY.
  2. You also can wrap all inputs with the "form" element so on submit when user clicks enter on the keyboard it will calculate results not only on the click handler.
  3. For class='input-flex' you can just have border-bottom property so it will have just border on the bottom.
  4. To align the button properly you can add position: absolute and to its parent on class='input-flex' position relative. With that you can place the button anywhere you would like with the properties: top, left, right, bottom.
  5. Semantically it is recomended to have only one h1 element on page and others you can make as h2s.
  6. Finally, you can have more margin with results and labels and also make text more bold.

  Wish you success with other projects.

  Marked as helpful

  0

  ---

  

  Eftimie Gheorghe•40

  @bigghitza96Posted 9 months ago

  @GioKhabu Thank you very much for your feedback, I will apply what you said on next projects I will take. Wish you success to.

  1