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

feat(sitewide): add scroll to top button to conditionally appear in longer pages #57

Merged

Conversation

DongreJaipal
Copy link
Contributor

@DongreJaipal DongreJaipal commented Mar 23, 2024

This PR Closes Issue closes

#28

Description

long blog posts there should be a scroll to pop button / icon that returns the user to the beginning of the page
for this app version primeicons used
applied in the app.component

What type of PR is this? (check all applicable)

  • 🅰️ Feature
  • 🐛 Bug Fix
  • 📝 Documentation Update
  • 🎨 Style
  • 🧑‍💻 Code Refactor
  • 🔥 Performance Improvements
  • ✅ Test
  • 🤖 Build
  • 🔁 CI
  • 📦 Chore (Release)
  • ⏩ Revert

Mobile & Desktop Screenshots/Recordings

[Attach screenshots or recordings if applicable]

Steps to QA

Added to documentation?

  • 📜 README.md
  • 🙅 no documentation needed

[Optional] Post-deployment tasks

[Specify any post-deployment tasks that need to be performed]

[Optional] What gif best describes this PR or how it makes you feel?

[Embed gif or describe the feeling in plain text]

1.renaming 
variable : enableScrollUp-showScrollButton
method : goPageTop-scrollToTop
2. hostlistener used instead of rxjs scroll evnt sub 
3.window.pageYoffet or document.scrollTop value  is used instead of cal scroll percentage.
4.space before ButtonModule
@esthersoftwaredev esthersoftwaredev changed the title feat(component): add new feature button that returns the user to the beginning of the page. feat(sitewide): add scroll to top button to conditionally appear in longer pages Mar 27, 2024
@esthersoftwaredev esthersoftwaredev merged commit 9d8c53e into HashnodeWithAngular:develop Mar 27, 2024
@esthersoftwaredev
Copy link
Member

@DongreJaipal thank you for your contribution 🚀

@DongreJaipal DongreJaipal deleted the my-fix-branch branch March 27, 2024 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants