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

  • Thomas 60

    @TomSif

    Posted

    nice work :)

    Marked as helpful

    1
  • Thomas 60

    @TomSif

    Posted

    Nice job, your work look almost identical to the model.

    One way to improve your code would be to use variable for better sustainability :root { --White: hsl(0, 0%, 100%); --Green: hsl(75, 94%, 57%); --Grey700: hsl(0, 0%, 20%); --Grey800: hsl(0, 0%, 12%); --Grey900: hsl(0, 0%, 8%); }

    And your HTML should contain a <meta> description tag that helps search engine determine what the page is about, Something like this <meta name="description" content="" />

    0
  • @vladzen13

    Submitted

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

    CSS Grid awesome even for small elements like userpic and username. Not only for page layout.

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

    I am not really happy with card on viewports smaller than 375px.

    It starts to shrink but slower than needed to prevent overflow. Could not find a solution in this iteration. Can somebody point me in right direction?

    Thomas 60

    @TomSif

    Posted

    First of all, congratulations on the quality of your work! I especially appreciate the use of var(--color) in your CSS, as well as the properties you’ve implemented to improve readability — they add a lot of value to your design.

    Regarding responsiveness and preventing overflow, you might consider using media queries and applying a quick animation on the tile. Personally, I used a transform: scale(0.8) on the tile to handle this issue. However, I must admit that I’m not entirely sure if this is the most appropriate method. It worked for me, but there might be a better solution."

    0
  • Thomas 60

    @TomSif

    Posted

    Great work, I like how you use "em" in your project, and using h3 and small was really smart. However, you could have used the "Outfit" font as shown in the style-guide and adjusted the background color to match the original. Nice job, well done :)

    0