Skip to content
OverviewSolutions
20
Comments
36

Aman Patil

@Amanpatil-DevNagpur, India885 points

My Name Is Aman . Love to work on New Technology and Deploy Projects. My Discipline: Do it once do it right .

Latest solutions

View more solutions

Latest comments

  • Aman Patil•885
    @Amanpatil-Dev

    Hello Hoang👋

    Your solution looks good and has all JS functionality as required👍

    -The issue that I found was in mobile width that is 425px your image of class header__imgContainer overflows and the hr line also. I have made a pull request on your github fixing this issue please do check it and merge if it looks appropriate to you.

    That's All Happy Coding👍

  • Aman Patil•885
    @Amanpatil-Dev

    Nice job Writam 👍 Looks good and responds well too

    Marked as helpful
  • Aman Patil•885
    @Amanpatil-Dev

    Hello Indra👋

    Your solution looks good and respond well nice job👍

    -The issue that I found was not an issue to be honest, but the reset button should be disabled till user inputs the number and then after clicking on reset button the reset button should be disabled again.

    That's All Happy Coding👍

  • Aman Patil•885
    @Amanpatil-Dev

    Hello Sebastian👋

    Your solution looks good and. Responds well 😀

    nice job👍

    -The issue that I found was in all the inputs field I as a user can type words, and negative values it should not be valid

    -Missing was Active States of input fileds and validation of No of people input field

    That's All Happy Coding👍

    Marked as helpful
  • Aman Patil•885
    @Amanpatil-Dev

    Hello Kiran👋, Nice job on this Challenge I really Like the Animation on Hover 👏

    Your solution responds well and Everything looks good to me 👍

    Just one thing i would like to correct although its not an issue at screen width 1600px your .column .card width increases what my recommendation is when we go above 1500px decrease the width of .column width of 22% would work I think.

    Thats all.

    Hope this helps✌️

    Happy coding😃

  • Aman Patil•885
    @Amanpatil-Dev

    Hello Mayura👋, Your solution responds well and Everything looks good to me 👍

    One advice I have is when making any design first what I do personally is scale down the browser width to 425px and design according to mobile first and then scale up for like 768px then 992px then 1200px what this do is make the design more mobile friendly scaling up would be easy because u just have to write media queries for scaling up

    For Example if you see your each card gets Skinner when you go to screen-width 801px till 1022px

    so your can write

    @media (min-width:801px){ .flex-container{ width:100% } }

    -Explanation:-we say that when you will go to Screen-width of 801px and greater than 801px apply width of 100% to class .flex-container

    then after scaling up to screen width of 1022px you can say

    @media (min-width:1022px){ .flex-container{ width:72% } }

    -Explanation:-Here we say that when you will go to Screen-width of 1022px and greater than 1022px apply width of 72% to class .flex-container

    Hope this helps✌️

    Happy coding😃

View more comments