Skip to content
OverviewSolutions
21
Comments
74

Aadvik

@Aadv1kDelhi, India1,250 points

Love learning tech; Open for collabs or open source projects.

I’m currently learning...

C and JavaScript

Latest solutions

View more solutions

Latest comments

  • Aadvik•1,250
    @Aadv1k

    Good, clean and polished. What else can one expect from such a challenge? Good work pal! I would have said mobile responsiveness but it looks great even on small screens

  • Aadvik•1,250
    @Aadv1k

    Hey! I don't have pro but attempted this challenge nonetheless (courtesy of you for the assets) Here is the github link live url

    The way I sovled the issue, was to just align all the images with a flex, justify them on the center and just translate the container by the width of a image on the arrow click

  • Aadvik•1,250
    @Aadv1k

    Holy hell I just saw the 40 html mistakes, but most of them seem to be the doings of tailwind and webpack, so I don't think I can fix em

  • Aadvik•1,250
    @Aadv1k

    this is because of the way your app is centered and the approach, this is why you should opt for a mobile first approach, on mobile do something like .card { width: 80%; max-width: 700px} and done, simple as that, no need to worry about the sizing, and also, when centering elements horizantally within the body use margin: 0 auto and use position absolute for the vertical centering, sorry if my comment is a bit vague, feel free to ask me anything

    Marked as helpful
  • Aadvik•1,250
    @Aadv1k

    hey man great design, just one small suggestion, (this is something even I do and am trying to avoid) that is hover, I would say you should add a :active animation to your button to make it clicky both on mobile and deskop. The problem with a hover is that it is sticky on mobile, which makes it irritating, in my newer projects I avoid using hover in most cases, especially where the effect is visible (box-shadow for instance) instead I try and add active animations that trigger when the button is clicked.

    Marked as helpful
  • Aadvik•1,250
    @Aadv1k

    Your design is great just a little suggestion -- add a media query for smaller screens like so -- @media only screen (min-width: 375px) {} and here you can maybe adjust some fonts and the width of the card.

View more comments