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

  • P
    Ian Parker 180

    @i-prkr

    Submitted

    What are you most proud of, and what would you do differently next time?

    I'm really happy with having spent the time to try to learn Sass properly, and implement it with the folder structure using partials. As well as this, to use Sass functions, maps and loops to be able to apply styling to the cards and change the grid based on media breakpoints. I also began to start using BEM which I plan to continue with.

    What challenges did you encounter, and how did you overcome them?

    I found it challenging understanding the best way to implement breakpoints in Sass, and it lead me to some great articles as linked in the readme that helped me understand a few Sass features.

    What specific areas of your project would you like help with?

    I would love to ask for feedback and help on two things in particular:

    1. My solution looks a lot bigger than the design in the screenshot. The style guide specifies a font size of 13px, but if I set this then the font does not appear as nicely on the design even though it is the correct size overall. So I kept it with the nicer font size, does that matter?

    2. Any feedback on my implementation of Sass and BEM would be awesome!!

    Thanks for looking at my solution!!

    Lackevil 80

    @Lackevil

    Posted

    Greate work! I like your stack. But preview site doesn't load styles.

    1
  • P
    Eli Silk 240

    @elisilk

    Submitted

    What are you most proud of, and what would you do differently next time?

    Thanks to the comments and looking at some other solutions, I was happy with my revised grid solution that takes advantage of the grid-auto-flow and the grid shorthand to position the cards correctly.

    What specific areas of your project would you like help with?

    If anyone knows how to get the top borders of each of the cards to have a straight edge rather than curved that would be helpful.

    Lackevil 80

    @Lackevil

    Posted

    You have greate vision!

    0
  • Lackevil 80

    @Lackevil

    Posted

    Good job! I like the hover animation on image. You should to create custom README, it gives others people to understand what you learned, what was your struggle and something else.

    0
  • P
    Eli Silk 240

    @elisilk

    Submitted

    I worked hard on getting closer to pixel perfection on this challenge, although I think in doing so I sacrificed a more general system of font sizes, margins, and padding. Would be interested in feedback about how I handled the margins and padding, and if there is a more general (rem/em) solution that might be more appropriate.

    Lackevil 80

    @Lackevil

    Posted

    Hello! Great job! I've seen your code and it's awesome. I suggest you to use BEM in your next projects.

    0
  • P
    Eli Silk 240

    @elisilk

    Submitted

    What are you most proud of, and what would you do differently next time?

    This challenge was fairly straightforward.

    What challenges did you encounter, and how did you overcome them?

    • I'm never sure if I've matched the font correctly.
    • I was confused about how the profile changed from the mobile to desktop, and so may not have captured that fully.
    Lackevil 80

    @Lackevil

    Posted

    Greate work!

    0
  • Lackevil 80

    @Lackevil

    Posted

    Hello! Greate work! Respect for rems and good organized.

    1
  • Lackevil 80

    @Lackevil

    Posted

    Hello, nice work! I suggest you trim the markup and styles and use semantics tags! Good luck!

    0
  • Lackevil 80

    @Lackevil

    Posted

    Nice work! Btw, try writing image sizes exactly in the HTML tag. It's good for optimization.

    0