Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

adding scroll to top button #361

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

Conversation

avdhendra
Copy link

@avdhendra avdhendra commented Jul 27, 2023

Related issue

Added Scroll to top button

  • (Add sorter for projects.json #100) Description
  • Added Scrollup button having icon of arrow up occurs when scroll value of y is greater 500
  • Scrollup button is a component inside component folder
  • having html,scss and js file for that component inside button folder

Testing instructions

Browser requirements (for page updates)

Your updates should work in the following environments:

  • Latest 2 versions of Edge
  • Internet Explorer 11 (should be useable, not pixel perfect)
  • Latest 2 versions of Firefox (one on Mac OS, one of Windows OS)
  • Firefox 68 (or latest version for Red Hat Enterprise Linux distribution)
  • Latest 2 versions of Chrome (one on Mac OS, one of Windows OS)
  • Latest 2 versions of Safari
  • Android mobile device (such as the Galaxy S9)
  • Apple mobile device (such as the iPhone X)
  • Apple tablet device (such as the iPhone Pro)

Ready-for-merge Checklist

Check off items as they are completed. Feel free to delete items if they are not applicable.

  • Expected files: all files in this pull request are related to one request or issue (no stragglers or scope-creep).
  • Browser testing passed.
  • Repository compiles.
  • Documentation (README.md, etc.) updated or added.
  • Approved by designer or writer (if applicable).

Merging

Please squash when merging and ensure your commit message uses conventional commit formatting.

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