@Vanillatte68
Posted
Good job on completing the challenge!
I think your positioning is good enough in desktop size, you only need to adjust the right
CSS property more.
I noticed the 'hidden' class for publisher toggle incorrectly when you open the share menu in desktop and then resize it to below 768px. You can adjust the event listener to toggle 'hidden' class for publisher when share menu is open and being resize.
I hope this will help. Happy coding
Marked as helpful
@MaanAlHababi
Posted
@Vanillatte68
The problem with adjusting the right property is that I can make it look fine on only one screen size, if I resize the window even slightly it gets messed up. There is a major difference between 1440px width and 1920px for example, I can only make it work for one which makes me believe that my method isn't optimal.
About the publisher section, I completely overlooked it! Thank you for pointing it out!