Age Calculator using Tailwind CSS and MomentJS

Community feedback

• 

  Bálint Balázsi•260

  @oroszlanoloPosted over 1 year ago

  Hi, congratulations on completing your challenge!

  I have some suggestions for you: First of all, I recommend structuring your code to different files. Now the index.html contains everything, but ideally, you would want to have a separate css and a js file. I can see, that the layout could be improved still. I have some suggestions for you:

  • Add a max-width to your instead of using different margins at different screen sizes. You could simply set a margin of like 2rem, and add a max-width of like 800px. This is a simple solution, and will work even on some ultrawide monitors.
  • I can see, that you are using font-weights, but they seem to differ from the design (your result text is way too thin) I think you should look into that.
  • Use bigger btn on big screen sizes. I like how you have set up a grid for the main layout

  I hope some of this was helpful. Have a nice day! :)

  Marked as helpful

  2

  ---

  

  Arya Wicaksana•20

  @wicaksana94Posted over 1 year ago

  Hello Bálint Balázsi,

  Thank you for your feedback. By the way I don't have ultrawide monitors, but your simple solution was helpful I'll update later :)

  Have a nice day too :)

  0
• 

  Andrew•130

  @FeelsGo-odPosted over 1 year ago

  Hi Arya!

  The calculator works like it should. But you can practice your css skills more by installing "pixel perfect" plugin on google chrome and making your layout look based on the design, so it will look like just the version on photoshop(or figma...).

  Tell me what do you think about my solution as well man. Thank you for posting yours!

  Marked as helpful

  1

  ---

  

  Arya Wicaksana•20

  @wicaksana94Posted over 1 year ago

  Hi Andrew,

  The pixel perfect plugin is new for me, I think it will help me better to design the layouts. Thank you, Andrew!

  1