Skip to content
OverviewSolutions
6
Comments
2
Victor De La Hoz
@victordlh29

All comments

  • Boulant Simon•20
    @WadbaWSubmitted 10 months ago

    My responsive in this project is very bad, but I did correctly the viewscreen required for the challenge (which were desktop Desktop: 1440px, Mobile: 375px). Without this issue, I think it works pretty well.

    FAQ accordion | Challenge

    1
    Victor De La Hoz•90
    @victordlh29Posted 10 months ago

    Hello, it's me here again, it's a pleasure to greet you, I saw your code again to continue helping you, I was surprised how good you already are with the class and the ID, you are evolving, keep it up, I liked the responsive, it needs to be polished more for other resolutions but it complies and is what is needed and expected for this challenge.

    0
  • Victor De La Hoz•90
    @victordlh29Posted 10 months ago

    Hello, What's up? I hope you are well, I really like your code, I was reviewing it and I find it interesting how you managed to meet the challenge but I will give you some recommendations so you can take them into account.

    In the part of the labels such as H1, P, etc., I recommend that you practice using classes or an id, for example:

    <H1 class="title">Improve your front-end skills by building projects</H1> <p class="description"> Scan the QR code to visit Frontend Mentor and take your coding skills to the next level</p >

    this will allow you to have more adequate and organized control of your code. I hope this helps you, so you can continue improving. ¡Good luck in your future projects!

    Marked as helpful
    0