Skip to content
OverviewSolutions
7
Comments
8
namborghinii
@namborghinii

All comments

  • namborghinii•220
    @namborghiniiPosted over 1 year ago

    Good job on completing the challenge! :)

    A reminder for next time: Group the number 1-5 by giving them the same name. By that the radio button will only be selected once.

    Marked as helpful
    1
  • Tamana•210
    @Tamana123-2Submitted over 1 year ago

    In this project, I had problem with the image's border radius although I made it the same as expected, but the border-bottom-left and border-bottom-right were getting extra range. Any recommendation regarding to the project would be appreciated, plus I have started Java script but I do not know how to go on except from (w3schools) can you recommend me any other fun and useful source? Thanks!

    HTML_CSS

    4
    namborghinii•220
    @namborghiniiPosted over 1 year ago

    Good job on completing the challenge!

    Instead of border-radius: 1.5rem 1.5rem 1.5rem 1.5rem; you can just write border-radrius: 1.5rem;and it will be valid for all corner.

    If you want to practice your JS Skills (or other languages) you should definetely check out codewars.com :))

    I hope I could help you. Happy Coding!

    0
  • namborghinii•220
    @namborghiniiPosted almost 2 years ago

    Good job on completing the challenge!

    Please make sure to include font-family in your CSS as well as using the right margin for all elements. A little reminder for the next time: Make a responsive Design with media queries or responsive layouts. :))

    HAPPY CODING!!

    0
  • Michał Karczewicz•70
    @MichalKarczewiczSubmitted almost 2 years ago

    I just finished my second challenge. I want your suggestions what can be improved and in general what you like or you don't like. Thank you in advance for feedback :)

    Product preview card component

    2
    namborghinii•220
    @namborghiniiPosted almost 2 years ago

    Good job on completing the challenge! :)

    Please remember to set font-family as well as font-weight for the different kind of sections.

    Marked as helpful
    1
  • Victor De La Hoz•90
    @victordlh29Submitted almost 2 years ago

    What did you find difficult while building the project? I found the css difficult I defend myself but I think I have to improve much more

    QR code component

    1
    namborghinii•220
    @namborghiniiPosted almost 2 years ago

    Just take more time for the challenge next time and you'll got it! :)

    Marked as helpful
    1
  • namborghinii•220
    @namborghiniiPosted almost 2 years ago

    Good job on completing this challenge so far! :) Little reminder to center the bottom text and decrease the scale of the component in general so it fits on the screen without scrolling. Also add media queries for responsive Design and a box-shadow and you should be fine.

    Marked as helpful
    2
  • namborghinii•220
    @namborghiniiPosted almost 2 years ago

    Good job on coding this even without the box shadow. Please pay attention to the text color next time! :)

    0
  • nzcastilan•40
    @nzcastilanSubmitted almost 2 years ago

    Had a difficulty of changing the image from mobile view to desktop view. I first tried to use srcset but can't seem to make it work so I ended up creating a div with a background image instead.

    Not sure also what is the best practice to use in this kind of scenarios. any feedback would be helpful :) Thanks!

    Product Preview Card using grid display

    1
    namborghinii•220
    @namborghiniiPosted almost 2 years ago

    Looks good so far! :)

    Please add a hover effect for the button and try to center the text and icon within the button field next time.

    Marked as helpful
    1