All comments

• 

  Tiago Martins•40

  @Tiago-Marts

  Submitted over 2 years ago

  

  Advice Generator App builded with Flexbox, JSON, JS and CSS

  2

  51
  

  Shashank Gupta•245

  @Shashank1003

  Posted over 2 years ago

  Hi Tiago! this solution looks good, you are handling long text nicely. But if you look at the small text on the small screen, it seems like the text is flying on the top with a lot of empty space at the bottom. To fix this, you can vertically align the advice text at the center (use flexbox) and give some margin-bottom to slightly uplift it.

  0
• 

  SAM•900

  @SAM-dev-pixel

  Submitted almost 4 years ago

  

  CSS and vanilla javascript

  2

  10
  

  Shashank Gupta•245

  @Shashank1003

  Posted almost 4 years ago

  Hi, you can add CSS {cursor: pointer} for the items that have some actions on click. It will improve user experience as rn I don't know which elements can be clicked, it just shows the default mouse pointer.

  0
• 

  Bashir Dauda•30

  @bashiribndawud

  Submitted almost 4 years ago

  

  Testimonial grid section

  2

  10
  

  Shashank Gupta•245

  @Shashank1003

  Posted almost 4 years ago

  Hey, hover animations aren't very smooth. Try to give at least 100-200 ms animation duration it will look better.

  0