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

[Panel] Tags Field: Selecting from Suggestions Broken in Safari #1646

Closed
neildaniels opened this issue Apr 2, 2019 · 3 comments

Comments

5 participants
@neildaniels
Copy link
Contributor

commented Apr 2, 2019

Describe the bug
Trying to select an autocomplete suggestion in the tags field does nothing in Safari. Works fine in Chrome.

      related:
        label: Related Pages
        type: tags
        accept: options
        options: query
        query: site.index

To Reproduce
Steps to reproduce the behavior:

  1. Have a tags field in a blueprint
  2. Type a few characters
  3. Try to click the result

Expected behavior
The clicked result should go into the field.

Screenshots
Screen Shot 2019-04-02 at 1 35 27 PM

Kirby Version
3.1.1 and 3.1.2-rc.1

Console output
None

Desktop (please complete the following information):

  • OS: macOS 10.14.4 (18E226)
  • Browser: Safari 12.1 (14607.1.40.1.4)
@texnixe

This comment has been minimized.

Copy link

commented Apr 2, 2019

Yes, I can reproduce this, you have to actually use the arrow keys to focus the option first, then click to confirm.

@bastianallgeier bastianallgeier modified the milestones: 3.1.3, 3.1.4 Apr 12, 2019

@distantnative distantnative modified the milestones: 3.1.4, 3.2.0 Apr 16, 2019

@distantnative distantnative modified the milestones: 3.2.0, 3.2.1 May 3, 2019

@luxlogica

This comment has been minimized.

Copy link

commented May 21, 2019

Just tested this on Firefox as well, and it is not a problem - seems to be a Safari-specific issue.

Unfortunately, it is also present in Safari on iOS.

bastianallgeier added a commit that referenced this issue Jul 1, 2019

@bastianallgeier

This comment has been minimized.

Copy link
Contributor

commented Jul 1, 2019

Safari has a different order in which it fires events. 🤬

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.