All comments

• 

  Ayoife•300

  @AyoifeSubmitted about 2 years ago

  Enjoyed solving this challenge!. An honest feedback would be helpful.

  

  Expenses Chart Component using CSS Flexbox

  1

  20
  

  Aaron Romanick•270

  @aaron-romanickPosted about 2 years ago

  You followed the design fairly well with your CSS so that it matches the design specs on both desktop and mobile! Great!

  A few things to keep in mind for next time:

  • I'm wondering why you changed the height of the bars in the spending graph. I know when you mouse over them, they change height to match the design docs, but I'm having difficulty figuring out your purpose. Also, the bars "jump" horizontally when you mouse over them so it makes it difficulty to mouse over.
  • I would maybe add a min-width to your overall container so that at very small sizes the design doesn't break.
  • I'd double check your semantic use of header elements; you have an <h2> before an <h1>, which 99.9% doesn't happen. Use your CSS to style heading tags, not the default browser styles.

  0

  This comment has received 2 repliesShow
• 

  Aldimhr•50

  @aldimhrSubmitted over 3 years ago

  Hello everyone, I made this with vite js + vue js. I need a lot of advice from you guys, feel free to say anything about this

  

  Calculator app with vite + vue

  1

  10
  

  Aaron Romanick•270

  @aaron-romanickPosted over 3 years ago

  Great start!

  One thing I noticed is that if you hit the "Reset" button, the theme toggle switch jumps to the #2 position unexpectedly.

  0