Skip to content

fix(Combobox): Chevron interactivity - #7975

Open
logiczsniper wants to merge 1 commit into
modrinth:mainfrom
logiczsniper:lczernel/combobox-chevron-pointer-events
Open

logiczsniper wants to merge 1 commit into
modrinth:mainfrom
logiczsniper:lczernel/combobox-chevron-pointer-events

Conversation

@logiczsniper

Copy link
Copy Markdown

Fixes #7927

  • Changes cursor when hovering over chevron
  • Prevents mousedown events to prevent the handleSearchFocusout handler from executing, which would cause another bug wherein clicking the chevron to close the dropdown would cause it to close (due to handleSearchFocusout) and then immediately re-open as the new handleSearchIconClick is called
  • Added handleSearchIconClick handler to enable opening and closing on click of the chevron

Note

This is my first PR in this repo - my first PR writing Vue in years 😅 and I don't see any test files... please lemme know if you wanna see anything else here 🙌

@github-actions

Copy link
Copy Markdown
Contributor

Pull request changelog

App

Added

Changed

Deprecated

Removed

Fixed

Security

Website

Added

Changed

Deprecated

Removed

Fixed

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

Security

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Version dropdown arrow icon is not clickable

1 participant