Skip to content
OverviewSolutions
4
Comments
4
P
cinquante six
@Cinquantesix

All comments

  • cinquante six•140
    @CinquantesixPosted 4 months ago

    Great job!

    You just miss the fonts. You can learn how to add it there: w3school

    good luck for the future!

    0
  • cinquante six•140
    @CinquantesixPosted 4 months ago

    Great job!

    If I can just share somethings with you because I make also this king of things and sometimes it begin to be difficult to see what's wrong:

    When you use child margin instead of parent gap, margin will overlap. Like your <a>. They have 10px margin top and bottom but they are only 10px from each other. I recommend to use a display: flex and gap: 10px instead. This is also very good to be sure to not add more space before and after your content like appens in your yolution.

    all the best for the future!

    Marked as helpful
    0
  • JJorgeMS13•440
    @JJorgeMS13Submitted 4 months ago
    What are you most proud of, and what would you do differently next time?

    Poder crear el diseño lo mas parecido posible, comentaria mas mi codigo.

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

    El poder hacer que la imagen se aclopara bien al tamaño de mi figure, lo logre con la propiedad object-fit: cover; y display: block;

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

    Me gustaria recibir retro de que es lo que puedo mejorar y si existe alguna manera de hacer que la propiedad de un elemento cambie si paso el mouse sobre otro elemento y que en los dos pueda hacer cambios sin necesidad de javascript.

    Tarjeta creada con HTML, CSS utilizando flexbox

    1
    cinquante six•140
    @CinquantesixPosted 4 months ago

    Hello,

    Your design is pretty good when you look the reference, congrats!

    In your code, you don't need to specify every size like width:…; and height:…;. Maybe you can read about how child and parent share some properties. For exemple, if you set a size for you .container-card, you can just push the image with padding and the content will automaticaly have is size (you just needed to adapt img).

    For the states changes, you can use pseudoclass like :hover you can read more about it there: CSS Pseudo-classes. There is a lot of states you can style.

    Good luck for the futur !

    0
  • cinquante six•140
    @CinquantesixPosted 4 months ago

    Hello,

    There is some useless tag maybe you can delete (all your id=circle*).

    If you want to add background like the circle or color, you need to have the qr code without is background.

    If you use ID ( <div id="circle1" > … </div> ), you can only use this ID once. If you want use something more than once, you need to declare a "class".

    Marked as helpful
    0