Design comparison
SolutionDesign
Solution retrospective
Hello there,
It still has lot to improve. Let me know if you could find any.
Thank you:)
Community feedback
- @CyrusKabirPosted about 3 years ago
hello my dear friend ♥ you did good and clean but here some advice :
- in mobile size you should use woman desktop mobile image and mobile pattern
- I don't know best practices about accordion in ux and ui but I think it's better user just be able open one of the accordion and when click to another .this one get close you know
- use an <main> tag to warp all content on it instead of an <article> or you can have <main> then for the child <article> but it's important to wrap all of your content and components to an one singe semantic element like <main>
Marked as helpful1 - @CrystalNyeinPosted about 3 years ago
I like the way the box moves on accordion open & close. I'm not quite familiar with transform translate yet. Still trying...
Btw Great work!
0
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