Skip to content
OverviewSolutions
9
Comments
5
Brijen Makwana
@BrijenMakwana

All comments

  • Brijen Makwana•160
    @BrijenMakwanaPosted over 1 year ago

    Você pode mudar o nome da função thanks(event) para algo como changeView ou showRating. Seria bom seguir os princípios de código limpo. E você pode seguir as mesmas regras de nomenclatura para estilos e variáveis. Ótimo trabalho.

    1
  • Brijen Makwana•160
    @BrijenMakwanaPosted almost 2 years ago

    Really well done. Try to use em or px for width, padding. Don't use rem for that, use rem for fonts only.

    Wish you luck.

    1
  • Brijen Makwana•160
    @BrijenMakwanaPosted almost 2 years ago

    Hey Mahek,

    Good attempt first of all. You can give border-radius of the image in px, because as you can see it's not affecting much on the top side of the image.

    And this is a really small example, so you don't need to provide multiple classes for tags. It'll crate confusion for the others. And since there is no javascript involved, you don't need to use id and class both for the tags.

    I wish you all the best.

    0
  • OPARA GOODNESS•10
    @imosfinestSubmitted almost 2 years ago

    Hello community, not sure if I completed this project perfectly. Please let me know what I should have done better. My first challenge was centering my container, not sure if I did that well..

    QR-code-component using html and css.

    2
    Brijen Makwana•160
    @BrijenMakwanaPosted almost 2 years ago

    Hey OPARA GOODNESS. Very good. My suggestion is to provide height and width of the images in the css file.

    0
  • Valarie Oyieke•60
    @ValarieOyiekeSubmitted almost 2 years ago

    I wasn't able to align the div with the prices at the center with display: flex; align-items: center; so I used align-items: baseline instead.

    Any help on what might have caused it not to work will be highly appreciated.

    Product preview card with Html and CSS

    3
    Brijen Makwana•160
    @BrijenMakwanaPosted almost 2 years ago

    Hey Valarie Oyieke, very nice. One suggestion from me is that to centre the .wrapper class, you can apply following styling.

    margin-left:auto;
margin-right:auto;

    Good luck for the next challenges.

    0