KiranAminPanjwani / MedStats

A Personal Health Journal Website
https://medstats-health-journal.netlify.app/
MIT License
30 stars 53 forks source link

added scroll to top button #132

Open pranavsuriya-sr opened 11 months ago

pranavsuriya-sr commented 11 months ago

🛠️ Fixes Issue : 131 and extra

👨‍💻 Changes proposed:

Added back to top button which is similar to the index page's back to top button.

✔️ Check List (Check all the applicable boxes)

📄 Note to reviewers

📷 Screenshots

image

netlify[bot] commented 11 months ago

Deploy Preview for medstats-health-journal ready!

Name Link
Latest commit 84fd85b3510684a39ae59462b18f1a85c00bf822
Latest deploy log https://app.netlify.com/sites/medstats-health-journal/deploys/65409a34a44ff30008d20e2f
Deploy Preview https://deploy-preview-132--medstats-health-journal.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

KiranAminPanjwani commented 11 months ago

Thank you for the contribution! ✨ However, the UI has been changed slightly. Please update your repository and adjust your changes accordingly. Your efforts in this regard would be much appreciated.

pranavsuriya-sr commented 11 months ago

Yes done now. Please review my changes and merge the pr. If there are some more changes needed, please get back to me. I'd be more than happy to help.

KiranAminPanjwani commented 11 months ago

"Back to top" button is missing in the sign up page.

pranavsuriya-sr commented 11 months ago

Please check now. I have added the scroll to top button to signup page too. View the deployed preview here: https://deploy-preview-132--medstats-health-journal.netlify.app/
Sorry for missing it out.

pranavsuriya-sr commented 11 months ago

Please review the changes and merge.

KiranAminPanjwani commented 11 months ago

Please re-check the changes you have made.

pranavsuriya-sr commented 11 months ago

Done