Design comparison
Solution retrospective
This is an insanely well designed challenge. It covers a lot of things. I cannot at all say that I did it well but at least I am happy with what I have come up with. I cannot wait to see how others have solved this because there would be millions of ways people can do this. Please, feel free to see my code and comment anything to help me improve my code.
Community feedback
- @vanzasetiaPosted over 1 year ago
Hi, Lungten Wangchuk! š
I strongly recommend having a simpler file structure. You do not need to have
src
anddist
folders. I recommend putting- the
index.html
on the root folder, - the
css
folder on the root folder, - the
app.js
folder on the root folder - and then adjusting the other things
This way, it is easier for people to give feedback on your HTML, CSS, and JavaScript code.
Here are some suggestions for improvements.
- All
<img>
elements must havealt
attributes. - Use the native HTML elements, the
<summary>
and<details>
elements to create the accordion panels. - Use a more modern CSS reset. My recommendation ā A Modern CSS Reset | Andy Bell
- Maybe create functionality where the users can only open one accordion panel at a time.
I hope this helps. Happy coding! š
Marked as helpful0@thelungtenPosted over 1 year ago@vanzasetia You're absolutely helpful. Thank you so much for taking time in reviewing my code. I will take in your suggestions for my next challenges.
0 - the
Please log in to post a comment
Log in with GitHubJoin 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