Article preview component

Community feedback

• 

  Miroslav•180

  @miroslavdurinPosted over 2 years ago

  Hello and congratulations on completing this challenge!

  I think that if you add this code to your media query should fix the issue:

  .soc-med {
          left: unset;
          top: 5rem;
          right: 0;
          transform: none;
      }
  

  I would also add a couple more suggestions:

  • Perhaps it would be a good idea to add event handler on window to close the social-media popup, something like this:

  window.addEventListener('click', (e) => {
      if(e.target.closest('.share-img')) return;
      if(!socMed.classList.contains('hidden')) socMed.classList.add('hidden')
  })
  

  • It would also make more sense to put img.share-img inside of a <button> element.

  I hope it helped you a little bit :)

  Marked as helpful

  2