Skip to content
Submitted almost 5 years ago

Article Card Preview | SASS, HTML, JQuery

Sam Buxcey•35
@sjbcreative
A solution to the Article preview component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Would be great to know peoples thoughts on the basic Javascript I have used to get the popup working, or if anyone has suggestion for how this whole part could be improved... Have I over-complicated this part of the task?

Also general Markup thoughts and feedback appreciated!

Thanks, S

Code

Please log in to post a comment

Log in

Community feedback

  • Sankara Gomathy M•455
    @sankaragomathym

    Hey! Nice work. There are few suggestions that I would like to give.

    1. It is always a good practice to load your script file at the end of the body as the DOM would have fully loaded and available for access in JS file.
    2. The share button in not visible in mobile view.

    Hope this helps.. :)

  • Abdulla Bashiroglu•520
    @bashiroglu

    Hi @sjbcreative, so your solution looks great. And using jquery for this task is okay. If you want to learn vanilla js, you can use dom manipulation. It is easier than jquery in this particular case. Good luck

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