Skip to content

Conversation

@nageshkharat
Copy link

Pull Request Template

Description

Added a Scroll Up button to the bottom-right corner of the website. This enhances user navigation by allowing them to smoothly scroll back to the top after reaching the bottom of the page. The button is styled similarly to the existing Scroll Down button and appears only at the end of the page.

Fixes # Enhancement idea for better UX (no open issue linked)

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

  • Manually tested on local machine
  • Verified that the button:
    • Appears only at the bottom of the page
    • Smoothly scrolls the user to the top
    • Matches the website's existing styling
    • Does not interfere with any existing functionality

Additional Context (Please include any Screenshots/gifs if relevant)

Here is a quick screenshot showing the Scroll Up button placement and styling at the page bottom:
scroll-up-screenshot

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules
  • I have checked my code and corrected any misspellings
  • I have tagged the reviewers in a comment below incase my pull request is ready for a review
  • I have signed the commit message to agree to Developer Certificate of Origin (DCO) (to certify that you wrote or otherwise have the right to submit your contribution to the project.) by adding "--signoff" to my git commit command.

Copilot AI review requested due to automatic review settings June 18, 2025 16:12
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a “Scroll Up” button to the bottom-right corner of the homepage to improve user navigation.

  • Renames the existing scroll-down container class to home-content__scroll_down.
  • Inserts a new scroll up button in index.html and adds corresponding CSS styling in main.css.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
index.html Updated scroll-down container class and added scroll up button markup.
css/main.css Renamed CSS selectors for scroll-down and added new styles for scroll up.

transition: all 0.15s ease-in-out;
}

.home-content__scroll_up {
Copy link

Copilot AI Jun 18, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The .home-content__scroll_up rule does not specify a bottom position, which may cause inconsistent placement of the button. Consider adding a bottom property similar to the scroll-down button to ensure it appears at the bottom-right as intended.

Copilot uses AI. Check for mistakes.
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you and congratulations 🎉 for opening your very first pull request in student-program

@nageshkharat
Copy link
Author

@Achanandhi-M Hey! I’ve submitted a PR for the Scroll Up button enhancement. Please take a look when you get time.

@pratik-mahalle
Copy link

Hey @nageshkharat, Thank you for raising the pr! Our team will look into it soon.
Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants