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

    @kaamiik

    Submitted

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

    This was a Grid challenge and It was a good one to use different grid properties.

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

    How to use different grid properties

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

    Please give me some feedbacks about this challenge. Anything you think can improve my code ability

    @BekzodIsakov

    Posted

    Event though testimonials section height is slightly taller compared to the original design, the solution look nice. I like how you used mixins to void code repetition

    0
  • @BekzodIsakov

    Posted

    It looks to me that the font-size of the main title is slightly smaller compared to the original design. Also card border should've been a bit thinner, maybe by 1px

    0
  • @BekzodIsakov

    Posted

    Looks nice! The one thing that's missing is that the image in the original design has a slight dark filter

    Marked as helpful

    1
  • @RubensTM

    Submitted

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

    Aumentaria as letras pra pode encaixar

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

    nenhum

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

    nehuma

    @BekzodIsakov

    Posted

    1. The card has much taller height compared to original design. 2. Second main issue is the layout of the right section of the card. It also needs to have some padding and spacing between inner elements. 3. Using a @mixin is a nice way to avoid repetition but it would be easier to understand and reuse a @mixin if its arguments has more descriptive names instead of a, b, c. 4. Also it's recommended that in scss nesting should not go beyond 3 levels.
    0
  • @BekzodIsakov

    Posted

    Looks nice! It turns out that our solutions and the source code are so similar. Good job!

    1
  • P
    Clyo 130

    @clyo

    Submitted

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

    Becoming more proficient in CSS

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

    For some reason the font-face doesn't work. It works in my preview on VS but doesn't work when I open the index.html file and look at it in the browser. Not sure why?

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

    For some reason the font-face doesn't work. It works in my preview on VS but doesn't work when I open the index.html file and look at it in the browser. Not sure why?

    @BekzodIsakov

    Posted

    Overall the solution looks good but it has some job to be done such as font-family, font-size, spacing between elements and the width of the card.

    0
  • @BekzodIsakov

    Posted

    Overall the solution look similar to the original design. But the spacing between elements and the width of the card have some work to be done

    0
  • @abigbroomstick

    Submitted

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

    I am proud that I was able to complete the challenge. I would try to avoid using when it's not necessary to build good semantic HTML habits

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

    I had some problems writing the CSS Flexbox & Grid as I haven't learned it yet so I had to read a few documentations and watch a couple of Flexbox & Grid explanation videos to write it.

    @BekzodIsakov

    Posted

    This solution includes semantic HTML, it is readable and has good accessibility.

    0