Skip to content
OverviewSolutions
5
Comments
5
milicaaa175
@milicaaa175

All comments

  • P
    ⭐⭐--Taylor--⭐⭐•130
    @Taylor-McNeilSubmitted 5 days ago
    What are you most proud of, and what would you do differently next time?

    Figuring out how to do the image breakpoints. The picture element was an interesting discovery.

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

    The responsive image changes. I googled around and discovered the picture element

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

    Is there a different way to accomplish the picture switching?

    For the button, how do you extend the button without padding?

    Product Preview Component Card

    2
    P
    milicaaa175•130
    @milicaaa175Posted 5 days ago

    Congrats on finishing the challenge, I think you did really good! There are only two things in your solution that are different from the design picture, but I think they are a quick fix:

    • More padding is needed around the text portion of the card
    • Color of the title is not right

    Good luck on your next project!

    Marked as helpful
    0
  • P
    milicaaa175•130
    @milicaaa175Posted 9 days ago

    Hi, congratulations on completing this challenge! Your solution looks really good! There are a few things I noticed when comparing it to the expected solution, so I will give my input here:

    • The top and bottom margins need to be a little larger, as well as the space between sections
    • Fonts, font colors and weights are a bit off in some places (mostly headings)

    One last thing, this web site is not responsive when changing the screen size, so maybe you could look into that.

    Good luck on your next project!

    Marked as helpful
    1
  • P
    milicaaa175•130
    @milicaaa175Posted 26 days ago

    Hey, good job on completing the challenge! I think you did really good! I noticed a few thing when looking at your solution:

    • Text containing the location should be bold.
    • I think that you need a bit more padding on link elements to make them look like the reference.
    • I got advice to use rem instead of px in my code, so there's that.
    • Another thing that I also got recommended and I tried it in this project is to use a proper CSS reset at the beginning of the stylesheet. I used this one CSS reset, so you can try it if you want.

    Good luck on the next project!

    Marked as helpful
    1
  • P
    OriD•120
    @oridrSubmitted about 1 month ago
    What are you most proud of, and what would you do differently next time?
    • CSS Clamp
    • CSS Nesting
    • wai-aria - ignore decorative images using alt="" role="presentation"

    Blog preview card

    2
    P
    milicaaa175•130
    @milicaaa175Posted about 1 month ago

    Hey, I think you did a really great job on this one! I see very useful things in your code that I didn't think of myself, so thanks for that! The only thing I see that is different from the original design is a bit more space after the author name, but I think that that's a quick fix.

    Marked as helpful
    0
  • P
    milicaaa175•130
    @milicaaa175Posted about 1 month ago

    Hey, good job on completing the challenge! I see some differences in your solution and the design, thought you might be interested to hear.

    • It looks like the text colors in both heading and paragraph are set to black, you can try and change them according to the style guide.
    • It also looks like the sizing of elements and their margins and padding is a bit off. You can try looking at the Figma file to see the exact measurements needed.
    0