Skip to content
OverviewSolutions
7
Comments
8
BekzodIsakov
@BekzodIsakov

All comments

  • Kamran Kiani•1,170
    @kaamiikSubmitted 5 months ago
    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

    Testimonials Grid Section

    1
    BekzodIsakov•100
    @BekzodIsakovPosted 3 months ago

    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•100
    @BekzodIsakovPosted 3 months ago

    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•100
    @BekzodIsakovPosted 3 months ago

    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
  • Rubens Carvalho•80
    @RubensTMSubmitted 3 months ago
    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

    position

    1
    BekzodIsakov•100
    @BekzodIsakovPosted 3 months ago
    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•100
    @BekzodIsakovPosted 3 months ago

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

    1
  • Clyo•140
    @clyoSubmitted 3 months ago
    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?

    Social Media Link Page

    2
    BekzodIsakov•100
    @BekzodIsakovPosted 3 months ago

    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•100
    @BekzodIsakovPosted 3 months ago

    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•70
    @abigbroomstickSubmitted 3 months ago
    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.

    Building a QR code component using HTML/CSS

    1
    BekzodIsakov•100
    @BekzodIsakovPosted 3 months ago

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

    0