Design comparison
Solution retrospective
I tried using only CSS in this one without JS .. all in all, it's an acceptable outcome but with minor issues that i hope to fix with your help
1 - Weird automatic scroll to the bottom of the page when clicking on the question ( tried preventDefault with JS didn't work )
2 - Background image moving when resizing the screen
3 - The image is also moving
If you find any other problem to mention or tips to give please let me know :D
Community feedback
- @abhineetkandelePosted over 3 years ago
Nicely done!!
Few comments from my end.
- Background image is covering a lot of area even on desktop view.
- When clicking a question, the whole card is sticking to the top of the page.
- Font can be imported directly inside the CSS file.
- And I would definitely recommend checking the accessibility issues in the report.
You could check out my solution using the below link. https://github.com/abhineetkandele/Frontend-Mentor-FAQ-Accordion-Card
It's not very great but I hope it may help.
In case, you feel something can be improved. Please do let me know.
1@DaoudAnas13Posted over 3 years ago@abhineetkandele thanks for taking time to mention the issues with details in this solution .. i'll make sure i'll work on fixing them ASAP :D
0
Please log in to post a comment
Log in with GitHub
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