Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • @jharteaga

    Posted

    Wow it's awesome!! Nice job!

    1
  • @jharteaga

    Posted

    Nice desktop design Stone! I would recommend you using media queries to build the responsive part for the mobile design :) you are almost done! great job!

    1
  • @clockwerkz

    Submitted

    I was really trying to align it to be pixel perfect; next time maybe I'll just spend the 5 dollars and get the design notes so I can nail down the exact spacing. :) Any comments or critique is more than welcome, thanks for looking!

    @jharteaga

    Posted

    Awesome work Carlos! You left it like the original design and very smooth responsiveness! There is a veeery little thing that I suggest you, maybe adding in mobile design a barely margin on the sides, because it's too close to the edges in some resolutions such as 440px. Excellent work though!

    2
  • @jharteaga

    Posted

    Excellent Bethany! I love your responsiveness as well. I just finished it but my responsive design needs a lot of improvements haha. I will take a look to your CSS to learn. Thanks for share your solution :)

    1
  • @jharteaga

    Posted

    Hi! I was reviewing your code, and in my opinion, some problems that I have identified are the following:

    • Combining positions (absolute and relative) with Flexbox.

    • Setting fixed width and height in some containers that is not necessary.

    • In mobile design, your flexbox alignment should be center instead of flex-start.

    • Using auto in right/left margin makes your card containers position too close from the sides without space.

    • Check the error in line 16 in your CSS file.

    Good going buddy! I hope these comments helps you :)

    1