-
Notifications
You must be signed in to change notification settings - Fork 561
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
fix: fixes scroll into view in custom container elements #677
Conversation
🦋 Changeset detectedLatest commit: 931744f The changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please follow the comment I mentioned below. Also please merge the latest master into your branch if that did not happen already.
Co-authored-by: Stef ❤️ <stefan.herz@sap.com>
* custom suggestions container revised to allow scrolling * changeset added * Update .changeset/smart-eyes-kiss.md Co-authored-by: Stef ❤️ <stefan.herz@sap.com> --------- Co-authored-by: Stef ❤️ <stefan.herz@sap.com>
Fixes issue: 674
What did you change (functionally and technically)?
When attempting to scroll through the custom container elements an error would occur. I found that the custom container was being rendered inside the unordered list instead of around it. Rearranged the elements
Checklist (remove this list before you submit the PR)
How to test:
maxHeight
andoverflowY: 'auto'
to the list styling in SuggestionsOverlay.jsup
anddown
buttons