question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

[v2] onMenuScrollToBottom does not fire if dropdown is not scrollable on mount

See original GitHub issue

@JedWatson I wasn’t sure how to test but you can see a codesandbox example here where opening the dropdown with options already loaded allows for the onMenuScrollToBottom callback to log “bottom” in the console. However if you clear the options before opening the dropdown, the callback will not fire after options have loaded.

I’ve submitted this PR as a fix.

Thanks!

Issue Analytics

  • State:closed
  • Created 5 years ago
  • Reactions:8
  • Comments:6 (1 by maintainers)

github_iconTop GitHub Comments

4reactions
macobocommented, Jan 25, 2019

@JedWatson What’s the next step to resolve this bug?

There’s two existing PRs addressing this usecase, neither with clear feedback on what needs to be done differently to accept the PRs:

1reaction
Methuselah96commented, Nov 8, 2021

I updated react-select to the latest in the CodeSandbox and I believe this issue has been resolved: https://codesandbox.io/s/interesting-microservice-dm8rk?file=/src/index.js. Closing this, feel free to comment if you think this is still an issue.

Read more comments on GitHub >

github_iconTop Results From Across the Web

javascript - Prevent scrolling of parent element when inner ...
This does not appear to be working in Chrome 34. The binding is firing, but the page continues to scroll when the <ul>...
Read more >
API - React Select
A flexible and beautiful Select Input control for ReactJS with multiselect, autocomplete and ajax support.
Read more >
Smooth Scrolling | CSS-Tricks
Here's the code to perform a smooth page scroll to an anchor on the same page. It has some logic built in to...
Read more >
overflow - CSS: Cascading Style Sheets - MDN Web Docs
The box is not a scroll container, and does not start a new ... scrollTop property may be used to scroll an HTML...
Read more >
scrollTo | Cypress Documentation
You can ignore the error checking to ensure the element is scrollable by passing ensureScrollable: false . // will move on to next...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found