Skip to content

Comments

Fixed a bug in which the selection was not made even if it was selected again.#38

Merged
Tsjippy merged 1 commit intobluzky:masterfrom
zaru:fixed-multiple
Jan 13, 2023
Merged

Fixed a bug in which the selection was not made even if it was selected again.#38
Tsjippy merged 1 commit intobluzky:masterfrom
zaru:fixed-multiple

Conversation

@zaru
Copy link
Contributor

@zaru zaru commented May 6, 2022

Thanks for the great library.

Fixed a bug that when the multiple option was enabled, it was not enabled even if the choice was selected again.

Steps to Reproduce

  • Set multiple option <select multiple="multiple">
  • Click on an option
    • document.querySelector('select').selectedOptions.length equals 1
  • Click on an option
    • document.querySelector('select').selectedOptions.length equals 0
  • Click on a choice and select again
    • document.querySelector('select').selectedOptions.length equals 0 // NG, expect 1

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