Skip to content
OverviewSolutions
0
Comments
4
Carl
@Carl256

All comments

  • Carl•45
    @Carl256Posted over 3 years ago

    well done mate, looks good so far, reduce the height of the card in your solution slightly as well as that margin below the p or above the button and it should look good.

    1
  • Satya Patra•30
    @arjunn21Submitted over 3 years ago

    I started learning Frontend just recently. Comments and suggestions are appreciated :)

    HTML CSS Bootstrap5

    1
    Carl•45
    @Carl256Posted over 3 years ago

    those cards need some height. In contrast to your solution, your solution is half the size of the design, however you doing great @Patra, you can do this.

    1
  • Lynette Lopez•30
    @lynettelopezSubmitted over 3 years ago

    I implemented the suggestions left on my last challenge: declared decorative images as background-image(s) rather than in HTML tags, used semantic HTML5 structural elements, and (tried to) use more heading elements.

    However, getting the orange floating box to appear on top of the component while declared as a background-image was tricky. Eventually, I just decided to remove it from my CSS and use an img tag instead.

    FAQ Accordion Card ft. SCSS

    3
    Carl•45
    @Carl256Posted over 3 years ago

    Hi, well done, the design looks great and is browser compatible. I have about 2 remarks to make though. 1 - slightly reduce the width and of the card to match the design. 2 - Hide/show the answer to a question when the question is clicked(does show but does not hide if I am still on the same question)

    1
  • Carl•45
    @Carl256Posted over 3 years ago

    Hey mate, seems like you forgot the shadows, And the card does not work well in safari. I am updating the version of os I have so that I can point you to the right direction

    0