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

[Package]: migrate ScrollToTop component to TypeScript #470

Merged

Conversation

raskolnikov-rodion
Copy link
Contributor

Description

Migrate the ScrollToTop component to TypeScript.

Issue: opentargets/issues#2871

Type of change

  • TypeScript refactor

How Has This Been Tested?

Applications were built locally. In the TargetPage, clicking in one of the items listed in the Molecular Interactions table reloads the page with new target data and scrolls the page to the top.

Checklist:

  • 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
  • Any dependent changes have been merged and published in downstream modules

@carcruz carcruz added the community Open source contributions from community label Sep 17, 2024
@carcruz carcruz merged commit 515a594 into opentargets:main Oct 8, 2024
6 checks passed
@raskolnikov-rodion raskolnikov-rodion deleted the refactor/scroll-to-top-ts branch October 10, 2024 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Open source contributions from community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants