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

Scroll To Top widget improved #655

Merged
merged 2 commits into from
May 31, 2024
Merged

Conversation

sau-mili
Copy link
Contributor

Description

improved the scroll to top widget

Fixes #580

Replace issue_no with the issue number which is fixed in this PR

Screenshots

Screenshot 2024-05-31 013035
ignore high contrast

Checklist

  • Tests have been added or updated to cover the changes
  • Documentation has been updated to reflect the changes
  • Code follows the established coding style guidelines
  • All tests are passing

Copy link

netlify bot commented May 30, 2024

👷 Deploy request for counsellor-startup pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit cf803d8

Copy link

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

Copy link
Contributor

deepsource-io bot commented May 30, 2024

Here's the code health analysis summary for commits 72d746f..cf803d8. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript✅ SuccessView Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

Copy link
Collaborator

@thestarsahil thestarsahil left a comment

Choose a reason for hiding this comment

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

Working Good !

@thestarsahil
Copy link
Collaborator

One more updation , update the navbar property on all other pages with same properties and working same as dashboard page !

@thestarsahil thestarsahil merged commit d1053f9 into Counselllor:master May 31, 2024
3 checks passed
@thestarsahil thestarsahil added gssoc for GSSOC'24 contributors level2 labels May 31, 2024
@sau-mili
Copy link
Contributor Author

Will do that tonight!

@thestarsahil
Copy link
Collaborator

Will do that tonight!

Sorry for the late reply did you made it ?

@sau-mili
Copy link
Contributor Author

sau-mili commented Jun 2, 2024

Which one? The navbar? I am on it right now

@thestarsahil
Copy link
Collaborator

yes !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gssoc for GSSOC'24 contributors level2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Enhance] scrolltotop widget
2 participants