Front end mentor challenge: Article preview component

Solution retrospective
Please tell me if anything can be improved- especially with the javascript!
Please log in to post a comment
Log in with GitHubCommunity feedback
- @dylan-dot-c
looking good! I saw you commented on someone's solution and they used the .toggle method which allows smooth switching of state with css
One thing I realized is that you should try and use only one popup container and try and make it work for both desktop and mobile screen sizes since you only have to write some extra css code and media queries, other than that you should be good
For semantic html, I see you use a section in a div, you could make the card a section or even an article and make the content inside a div if you want
Marked as helpful
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