Skip to content
OverviewSolutions
3
Comments
5
Haris Akbar
@Har1s-Akbar

All comments

  • Haris Akbar•100
    @Har1s-AkbarPosted about 2 years ago

    Nice try there are few things i would like to point out in your mobile version. It seems you are using max width for the text on mobile version too, you should use 60%-50% width on text with m-auto to center it. if you want to center it without reducing the width you should use text center css property. I would suggest you decrease the text-size, text width on mobile version with text center. I would also suggest you to use tailwind css because it rids you of trouble of writing the css in separate file.

    0
  • Haris Akbar•100
    @Har1s-AkbarPosted about 2 years ago

    Well done you did a great job but there are some issues I see in the design.

    1. In mobile version I suppose you are using height screen which is not a best practice, it should be height full. 2)In desktop version you should use width full in that way your design will span across whole screen.
    Marked as helpful
    0
  • Haris Akbar•100
    @Har1s-AkbarPosted about 2 years ago

    You have done a great job. There are somethings wrong with the mobile version, I think you are using height to the screen because it is not scrollable on mobile version and also the background is not covering the whole width of screen on desktop version. You should fix these issues. All in all you did a great job, keep coding

    0
  • Haris Akbar•100
    @Har1s-AkbarSubmitted about 2 years ago

    Hello everyone this is my second project on front end mentor. The challenge was easy peasy but doing transition gave me a headache and I ended up skipping the transition. I always struggle at transitions

    Advice-generator-app

    1
    Haris Akbar•100
    @Har1s-AkbarPosted about 2 years ago

    I figured out the way to add the transition thanks everyone

    0
  • Haris Akbar•100
    @Har1s-AkbarPosted about 2 years ago

    Good Job, well done I think there are some of the things you might have done better 1)Like the divider is a bit large it is drawing more attention than necessary and the font of advice is also a bit large.

    All in all well done

    1