Skip to content
OverviewSolutions
8
Comments
3

Adriana Eka Prayudha

@radenadriBandung, Indonesia160 points

Currently trying to tackling some frontend challenges after staying too long with backend stuff. Trying to embrace the CSS!

Latest solutions

View more solutions

Latest comments

  • Adriana Eka Prayudha•160
    @radenadri

    Hello dngtnv, congratulations it's a great work! 👏🙌

    The layouts and the elements are match the design, and the dark mode toggle works as well!

    But i found one specific issue and one addition based on my opinion, especially in mobile (<= 425px) viewport :

    1. The top padding on social-media container are blocking the dark mode toggle, so the toggle can't be clicked. I believe the solution for this you can add margin-top to this container on mobile breakpoint, maybe 40px is enough and you good to go!

    2. Since the user id element contains image and text, i think you can use span element to wrap the elements, because span acts like a div but for inline element, and for the text you can wrap it in p tag. Because p tag mostly used for text only. 😁

    Reference : https://developer.mozilla.org/en-US/docs/Web/HTML/Element/span

    That's it from me, keep coding! 👏🙌

    Marked as helpful
  • Adriana Eka Prayudha•160
    @radenadri

    Great work man! The layout works great & the animations run smoothly in every device width! 🙌

    I have several additions based on my opinions :

    1. If you increase the line height on the text a little bit, say it at least 2 or 2.15 i think it'll be great 😃.
    2. The button hover state works perfectly! But if you also change the text color to match the card background color, that would be more match the design as well!
    3. There's a horizontal overflow on smaller width, you might wanna check what's going on 😊.

    Overall it's a great work, congratulations! Happy coding and also keep coding man! 👌👏

  • Adriana Eka Prayudha•160
    @radenadri

    Nice work man! It works greatly on tablet and mobile as well! 🙌

    From what i see, i think i only have two additions based on my personal opinions :

    1. It will be much more awesome if you add the box-shadow on each testimonials, just a thin shadow to make it look exactly like the design. But overall its great, its only little additions as well. 😊
    2. I believe the provided design showing that the avatar is kind of small, if you change it to approximately 45px, i think it will be much more great since it will be much more clearer to see the picture.

    That's it, great work for sure! Happy coding and also keep coding man! Waiting for another work! 👏👌

    Marked as helpful