-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Inconsistent delete behavior #2521
Comments
Thanks for the detailed report, @ramdog ! However, I can't reproduce 😬 What browser, browser version, OS, and OS version are you using? |
@tjschuck Thanks for the quick reply! Good point, should have noted:
|
Hm, on the same version of Chrome, I'm not seeing the issue. If I press cmd-backspace, as soon as I release command, it clears as I'd expect. I'm on OS X El Cap, though, so perhaps that's the problem, though I doubt it would be... |
Hmm, I just borrowed my cubemate's machine which is on El Cap (not sure on Chrome version) and I saw the same behavior (the buggy behavior). When you say you can't reproduce, does that mean you only see the results in Flow A or Flow B? |
Same as your expectations -- Flow B seems correct; Flow A seems like a bug. But for me, Flow A (cmd-backspace) also clears the search as expected. |
Got it, interesting. One thing I've noticed (without looking at the source yet) is that there seems to be some sort of debounce when updating the
then I don't see the search update from: to
until I let go of delete. Just a random observation, not sure if that's related to the issue. In any case, if you'd rather close this issue, that's cool. |
I'll leave it open for now for someone else to hopefully confirm/deny a reproduction on their end as well. Please submit a PR if you have any ideas on getting the behaviors to match! |
Will do, thanks. Last thing to note for anyone looking into this: it seems I get the expected (Flow B) behavior when I use |
I can reproduce. Probably the keyup event of the meta key is not handled. |
I can reproduce. This Stackoverflow thread has details of the issue. It is expected behaviour, with no known workaround to the root issue. I've submitted a PR which simply allows the search function to run when Cmd is pressed. This is an effective workaround, and doesn't seem to introduce any issues (other than potentially being less efficient if someone is pushing Cmd/Windows Key). |
There is inconsistent behavior with respect to deleting a search. Steps to reproduce:
Flow A:
Flow B:
I would expect it to always work as Flow B. Flow A seems like a bug to me.
Screencast taken directly from demo page:
![ulxzkbt7dc](https://cloud.githubusercontent.com/assets/5456178/12799112/fb85b072-ca81-11e5-9dee-5fdada0d8d40.gif)
The text was updated successfully, but these errors were encountered: