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

Added mouse selection support, add the ability to display a drop-down box even if there is no content in the input box #23

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Lemon-2333
Copy link

Here is a rough outline of the content changes I've provided to the parent repository:

Add content

  1. Added the always_display parameter to the Dropdown component to ensure the dropdown box is visible even when the text box is empty but in focus.
  2. Implemented the ability to select dropdown box contents using the mouse.

There are some points that need discussion:

After adding always_display, I opted to modify the _input_focus_changed function for its implementation to enhance flexibility. However, I have concerns regarding potential performance impacts, so I'm considering whether to transition this functionality to the Input detection within the on_mount function.

The new parameter always_display is used to control the drop-down box displayed when no content is displayed in the input box. The default value is False
Added mouse control, now you can select complete items with the mouse
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.

1 participant