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

Disallow multiple dropdowns #17083

Open
wants to merge 4 commits into
base: gh-pages
Choose a base branch
from

Conversation

saintsGrad15
Copy link

@saintsGrad15 saintsGrad15 commented Jul 14, 2017

To reproduce the error:

  1. In a text box with awesomeplete enabled, choose an enumerated value.
  2. Blur the text box
  3. Focus the text box, displaying the drowdown.
  4. All without the mouse, select some portion of the text (even all of it if minChars == 0) and delete the selected text
  5. Observe that a second box appears but quickly covers the original box.
  6. Click one of the options closing the box.
  7. Observe that another box is still present.

Sometimes there are several although I can only reliably reproduce a two-box scenario.

See screenshot below. This was observed in:
FF 52.2.1
Chrome Canary 61.0.3150.0
Chrome 59.0.3071.115

See below for a screenshot of the developer console.

image

In addition a minor style typo was corrected and a Webstorm (Jetbrains IDE) hidden directory was added to .gitignore.

@LeaVerou
Copy link
Owner

Hey, thanks for contributing!

If calling this.open() is a problem when the dropdown is already open, shouldn't the isOpened check be in the open() method itself?

@ghost
Copy link

ghost commented Oct 23, 2020

Experiencing the same issue with Turbolinks and Ruby on Rails. Any update on this?

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

Successfully merging this pull request may close these issues.

2 participants