All page content should be contained by landmarks
<div class="attribution">
Challenge by
<a href="https://www.frontendmentor.io?ref=challenge" target="_blank">Frontend Mentor</a>. Coded by <a href="#">Florin Porut</a>.
</div>
Learn more Looking to hire developers?
<div class="attribution">
Challenge by
<a href="https://www.frontendmentor.io?ref=challenge" target="_blank">Frontend Mentor</a>. Coded by <a href="#">Florin Porut</a>.
</div>
Learn more It has been a while since my last challenge on this platform, I try my best not to slack off to much. If anyone can tell me how to make the pop-up stay in the same place for big screens I would appreciate it.
Feedback is always welcome!
@kenreibman
Posted
Hey! Great submission!!
Instead of having share__options
as a child element of your share__section
, I would have it as a child element of your main
content. That way your position absolute would be within that container, making it easier for you have the same position.
I hope this helps!
Marked as helpful
@TheCoderGuru
Posted
Hey good job completing this challenge.
Keep up the good work
Your solution looks great however I think the background of the body element needs to be light blue secondly I think the content on the right side of the card has a bit too much margin on the left
I hope this helps
Cheers Happy coding 👍
Join over 180,000 people taking the challenges, talking about their code, helping each other, and chatting about all things front-end!