fix: move close button to top of share modal for better UX

[maaviah17]

📘 Pull Request Template – PhysicsHub

Thank you for contributing to PhysicsHub!
Please complete the sections below to help us review your pull request efficiently.

---

🔍 Description

This PR improves the usability of the share modal by moving the close button to the top of the modal container. Previously, the close button was placed after the social media icons, requiring users to scroll before being able to dismiss the modal.

Changes Made

• Moved the close button to the modal header section
• Improved layout alignment for better visual structure
• Added a dedicated header container for cleaner component organization
• Ensured the close button is easily accessible without scrolling

Benefits

• Better user experience and accessibility
• Faster interaction for users, especially on smaller screens
• Follows common modal UI/UX conventions
• Improves overall layout consistency

Closes #191

---

✅ Checklist

Before requesting a review, please ensure that you have:

• Verified that the project builds and runs locally (npm run dev)
• Ensured no ESLint or TypeScript warnings/errors remain
• Updated documentation, comments, or in-code explanations where needed
• Verified responsiveness across devices (desktop, tablet, mobile)
• Followed the CONTRIBUTING.md guidelines

---

🎨 Visual Changes (if UI-related)

Before

After

---

📂 Type of Change

• 🐛 Bug fix (non-breaking change that fixes an issue)
• ✨ New feature (non-breaking change that adds functionality)
• 💥 Breaking change (fix or feature that would cause existing functionality to change)
• 📝 Documentation update
• ♻️ Refactor / code quality improvement
• [x ] 🎨 UI/UX enhancement
• 🔒 Security improvement

---

🧩 Additional Notes for Reviewers

[maaviah17]

Hi, I’ve implemented the requested changes and attached screenshots for reference.
Please let me know if any modifications are needed. Thank you!

[physicshub]

Hey, sorry for late reply, your PR is fine and almost ready to be merged. Unfortunatly while trying to delete the package-lock.json from the PR, I accidentally deleted it entirely. Can you please update the PR so that these changes aren't recorded?

After this I'll merge your PR

[maaviah17]

I’ve restored the package-lock.json file and updated the PR.
Please let me know if anything else is needed. Thanks!

[mattqdev]

Thank you so much for this fix!