Skip to content
OverviewSolutions
5
Comments
4
Brad Proctor
@MrBradley815

All comments

  • Brad Proctor•70
    @MrBradley815Posted 3 months ago

    This looks really good! The only major difference I can see is the height difference. The design has large padding top and bottom that is throwing your solution off height-wise.

    1
  • Brad Proctor•70
    @MrBradley815Posted 3 months ago

    Really only needs a few small tweaks. I would just make sure those links are bold.

    The only other thing I would say is to try and use more appropriate html tags. You've used divs for most everything. Try and use more semantics such as <nav> or <main>

    0
  • Carmo Lourenço de Souza Batista•50
    @Carmo22bSubmitted 3 months ago
    What are you most proud of, and what would you do differently next time?

    Não sabia que no box-shadow podia deixar a sombra muito em negrito, achei esse estilo de card diferente. De diferente que eu faria era adicionar uma cor gradiente combinando com o fundo amarelo e a sombra do card.

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

    Os desafios que passei foi acertar o tamanho da imagem do card com os texto em baixo, mas com a ajuda do chatGPT, eu consegui ajustar o tamanho.

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

    Queria entender melhor sobre como funciona o "transform" e o "translate" na parte do CSS.

    box-shadow, display: flex, width, border-radius, background-color

    3
    Brad Proctor•70
    @MrBradley815Posted 3 months ago

    This looks really close! Only thing I would watch out for is your font. Your font sizing and line heights are a little off. Straighten those out and this will be great!

    0
  • Brad Proctor•70
    @MrBradley815Posted 3 months ago

    The one difference that probably stands out the most to me is the font family. Just in case you didn't know, the font for this project is available on Google Fonts.

    Your HTML markup overall looks pretty good. One thing I would recommend is for the bold text to use a heading tag.

    The sizing of your component is almost spot on. The only real issue being that almost all of your widths are hard coded. Generally you will want to avoid using hard set width/height values like that. Try to stick to a more responsive style so that it will scale properly to larger and smaller screens. An example would be instead of using width, try max-width on your main content div. It won't exceed that value and if the container that this component is ever in is shrinks, it will scale properly without breaking.

    Marked as helpful
    0