Skip to content
OverviewSolutions
20
Comments
29
Art
@Art-wdt

All comments

  • Art•420
    @Art-wdt

    Maybe it doesn't matter.

    1. About this - check your report https://www.frontendmentor.io/solutions/responsive-landing-page-using-css-flexbox-f7wwnBubd/report

    2. Please see design files in your challenge. I think .testimonials{background-color: rgb(255, 251, 248);} for example

  • Art•420
    @Art-wdt

    Hi @brodiewebdt ! I completed this challenge just now.

    I have no words. I liked your work. Great job. Excellent burger menu

    Wish you happy coding!

    Marked as helpful
  • Art•420
    @Art-wdt

    Hi @DeCaptainDan ! I completed this challenge just now. I liked your work. Great job

    What about:

    1. HTML5 Landmark Elements
    2. Background color of the section with reviews
    3. <ul> must directly contain only <li>
    4. Burger menu doesn't work

    Wish you happy coding!

  • Art•420
    @Art-wdt

    Hey! Great job! Keep coding! Improve your web developer skills

    Where are your answers to questions? Answer # 1, Answer # 3-5 are missing

    I also did this and used jQuery. I will be glad to your opinion. Please see my work.

    Happy coding!

  • Art•420
    @Art-wdt

    Hey! Great job! Keep coding! Improve your web developer skills

    You can make background-color:white; for FAQ`s block.

    Heading bolder when click that will be perfect.

    Also, when every answer are opened, it is makes the design more boring. You could have either:

    1. Make the block height larger. to contain everything when too many menus are opened. or
    2. Open only one answer at a time. When opening the next one, close the previous one.

    I also did this and used jQuery. I will be glad to your opinion. Please see my work.

    Happy coding!

    Marked as helpful
  • Art•420
    @Art-wdt

    Hey! Great job! Keep coding! Improve your web developer skills

    You can make arrow rotate (180deg)

    Also, when every answer are opened, it is makes the design more boring. You could have either:

    1. Make the block height larger. to contain everything when too many menus are opened. or
    2. Open only one answer at a time. When opening the next one, close the previous one.

    I also did this and used jQuery. I will be glad to your opinion. Please see my work.

    Happy coding!

  • Art•420
    @Art-wdt

    Hey! I love your job! Looks great!

    But what about keeping your block height fixed? Now the picture of the background is driving parallel to the answers. Maybe I'm wrong.

    In any case, your work looks cooler than mine. I also did this and used jQuery.

    Happy coding!

  • Art•420
    @Art-wdt

    Hey! @PHnrq

    Great job! I like it.

    But what about keeping open only one question? Not all questions would be opened. Close previous, open new. The height will not increase.

    I have completed this challenge. Maybe not as great as you are. I used jQuery.

    Happy coding!

    Marked as helpful
  • Art•420
    @Art-wdt

    Hey! Great job! I like it.

    But what about keeping open only one question? Not all questions would be opened. Close previous, open new. The height will not increase.

    Or you can use overflow:scroll;

    I have completed this challenge. Maybe not as great as you are. I used jQuery.

    Happy coding!

    Marked as helpful
  • Art•420
    @Art-wdt

    Hey! Great job! I like it.

    But what about keeping open only one question? Not all questions would be opened. Close previous, open new. The height will not increase.

    I have completed this challenge. Maybe not as great as you are. I used jQuery.

    Happy coding!

  • Art•420
    @Art-wdt

    Hey @ApplePieGiraffe ! Always, you surprise me with the level of your skill. This is probably one of the best accordion challenges on this site. I will learn a lot from your example.

    How about making it look great on a screen less than 310px wide? Maybe I'm wrong. Perhaps my question is stupid. Thanks anyway for your great example!

    Happy coding!

  • Art•420
    @Art-wdt

    Wow! Pixel perfect! I am very surprised by your work!

  • Art•420
    @Art-wdt

    Hi! Very good for destop and laptop. But where is your mobile version of the site. For tablets and mobile devices?

    You can add media queries to your stylesheet. For various device screen widths

    Happy coding

  • Art•420
    @Art-wdt

    Hey! I was pleased with your work! You completed this task better than me!

    Original responsive design!

    But take a look at the button design. At the time of the screen width 850 - 1000 pixels. It's practically a circle.

    In the end, I will say that your work is excellent. I wish you success.

    Happy coding!

  • Art•420
    @Art-wdt

    Hey! Nice try. But at the moment I do not see the svg files. Please change the path. Or create an images folder.

    Also you can create a mobile version of the site. Use media queries in your stylesheet (style.css)

    Happy coding!

  • Art•420
    @Art-wdt

    Hey @foolhardy21 ! Great job!

    I also did this challenge recently. I just like you have an error with the background. See design files. The background should be gray, not white.

    I think you can add the mobility of the screen width from 500px to 1000px

    Happy coding!

  • Art•420
    @Art-wdt

    Hey, @MarkusKeane Great job! I love your solition! The only thing I can say. How about additional media queries in your style? For example, from 800 pixels to 1350 pixels. Your blocks have no empty space between each other. You can add media queries to your CSS

    It was also my first job on this site. I did it not so long ago. Happy Coding!

  • Art•420
    @Art-wdt

    Hey, @fernandomad Great job! The only thing I can say. How about the mobile version? You can add media queries to your CSS

    It was also my first job on this site. I did it not so long ago. Happy Coding!