Skip to content
OverviewSolutions
4
Comments
5
Mikaias Silva
@mikayas

All comments

  • Mikaias Silva•160
    @mikayasPosted over 1 year ago

    you must remove "image/" from src in img tag

    0
  • Holo Olagoke•50
    @Olarra18Submitted over 1 year ago

    This is my second complete challenge. Though, the image of the challenge seen to be simple but I find difficult to reduce the entire card height in desktop view.

    /*Desktop & Tablet view*/
@media screen and (min-width: 768px) {
.container {
max-width: 50%;
}
.img-box, .txt-box {
width: inherit;
height: inherit;
}
.img {
width: 100%;
height: 100%;
}
body {
margin-top: 2%;
}
    
}

    Any suggestion on how to improve on it are highly welcome, Thanks.

    QR-code-component-Challenge

    2
    Mikaias Silva•160
    @mikayasPosted over 1 year ago

    the css property "max-width" defines a maximum width for an element. Use in "img", max-width: 360px

    Marked as helpful
    1
  • MarianaSantos•10
    @MarianastefanySubmitted over 1 year ago

    O que você achou difícil ao construir o projeto? muito bom, achei ele bestante dinâmico e simples . Quais áreas do seu código você não tem certeza? na parte das div Você tem alguma dúvida sobre as melhores práticas? não

    Solução de componente de código QR - Frontend Mentor

    2
    Mikaias Silva•160
    @mikayasPosted over 1 year ago

    you always must name your main html to index.html. Then report the solution on this platform

    0
  • Mikaias Silva•160
    @mikayasPosted over 1 year ago

    you should use max-width to set a width limit for the card. I also recommend studying about css flexbox, will help with positioning elements.

    Marked as helpful
    0
  • Mikaias Silva•160
    @mikayasPosted over 1 year ago

    congratulations on the resolution... But i recommend you use flexbox or position: absolute to center the card on the screen. search about both.

    Marked as helpful
    0