Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Mobile first using only HTML and CSS [!JavaScript]

KTβ€’ 390

@KtGitIt

Desktop design screenshot for the FAQ accordion card coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Greetings AllπŸ‘‹πŸΌ,

Any feedback on anything will highly appreciated. πŸ™ŒπŸΌ πŸ‘πŸΌ

Javascript is not used because HTML's details tag provided functionality.

Images' alignment were challenging but I managed to get it done without adding them into HTML.

Keep coding, KT

Community feedback

Ahmed Faisalβ€’ 5,095

@afrussel

Posted

Wow πŸ‘ great work. So close to design. Found some issue on JavaScript. All option can open at a time. Please check accordion option

2

KTβ€’ 390

@KtGitIt

Posted

@afrussel Thanks for feedback. Javascript is not used at all for this. I tried to accomplish bonus point in challenge that stats the complete without using javascript. I am planning on doing it with javascript as well.

0
John_diddlesβ€’ 470

@Johndiddles

Posted

wow, this is really nice! One issue I noticed is that the height of the container changes when you open an accordion. Also when all accordions are open, I noticed that the bottom border of the last accordion cuts through the text in the footer. Not withstanding, you've done a great job.

I have done this challenge before. You can view it here https://johndiddles.github.io/faq-accordion-page Your comments are welcome

1

KTβ€’ 390

@KtGitIt

Posted

@Johndiddles This is great feedback. I fixed the footer issue. Regarding the change of height when accordion opens, I looked at your solution and you have same issue on mobile layout but not on desktop, but on desktop when all are open, top and bottom are almost touching to the container wipes out all the margin. Maybe design is meant not to have open all accordion at the same time. One opens and the other one closes at the same time. May be it is possible with JS. I haven't use any JS on this but planning on working on one with JS and I think that would fix height change issue. BTW, great job on your solution and animation looks great as well

0
Lincoln Dubeβ€’ 85

@slickLink

Posted

Wow just wow, I love your solution, I had never heard of html details tag before i came across your solution. I finished mine using :target but I couldn't get it to close after it was open unless you opened a new question. Honestly I can't give feedback to you but just wanted to say great job, I learned a lot from you solution, will definitely try using some in later challenges.

0

KTβ€’ 390

@KtGitIt

Posted

@slickLink Thank you for your feedback and I am glad that my solution was helpful.

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord