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

[JumpLinks] - support scroll container node reference for spy #9636

Closed
christianvogt opened this issue Sep 18, 2023 · 3 comments · Fixed by #9961
Closed

[JumpLinks] - support scroll container node reference for spy #9636

christianvogt opened this issue Sep 18, 2023 · 3 comments · Fixed by #9961
Assignees
Labels
Milestone

Comments

@christianvogt
Copy link
Contributor

christianvogt commented Sep 18, 2023

Is this a new component or an extension of an existing one?
What is the existing component, if any?
JumpLinks

Describe the feature
A clear and concise description of the new feature. What is the expected behavior?
JumpLinks has a prop called scrollSelector of type string whose purpose is: Selector for the scrollable element to spy on. Not passing a selector disables spying.

It should be possible to supply a direct node reference or call back function instead of only supporting a selector string.

Are there visuals for this feature? If applicable, please include examples for each state and for varying widths
Include screenshots or links to Marvel or other mockups.

None

Any other information?

Copy link

stale bot commented Nov 18, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the wontfix label Nov 18, 2023
@christianvogt
Copy link
Contributor Author

Bump waiting on response.

@stale stale bot removed the wontfix label Nov 20, 2023
@tlabaj tlabaj added the Pinned label Nov 30, 2023
@tlabaj
Copy link
Contributor

tlabaj commented Nov 30, 2023

We will address this issue in a future sprint.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants