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.

Sign up for GitHub

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

Jump to bottom

Choosing an operation name from the drop-down immediately after a page load does nothing, but works afterwards #150

Closed
dahjelle opened this issue Jul 21, 2016 · 0 comments
Closed

Choosing an operation name from the drop-down immediately after a page load does nothing, but works afterwards #150

dahjelle opened this issue Jul 21, 2016 · 0 comments

Comments

@dahjelle
Copy link
Contributor

If I have two queries in the editor, and try to run them via the drop-down on the execute button, nothing happens the first time I try after a page load, but it works fine afterwards.

The appears to only be an issue with multiple queries and using the execute button.

I attached some logging to the onMouseUp handler on the li, and it does not trigger. On the other hand, the _onMouseUp handler that _onOptionsOpen attaches does trigger.

It looks like the issue is that the _onMouseUp handler in _onOptionsOpen gets triggered first, runs setState, and hides the menu before the onMouseUp event handler is called.
dahjelle added a commit to dahjelle/graphiql that referenced this issue Jul 21, 2016
@dahjelle
Fixes graphql#150.
1ca2dbe
@dahjelle dahjelle mentioned this issue Jul 21, 2016
Merged
dahjelle added a commit to dahjelle/graphiql that referenced this issue Jul 21, 2016
@dahjelle
Fixes graphql#150.
75198c6
dahjelle added a commit to dahjelle/graphiql that referenced this issue Jul 26, 2016
@dahjelle
Fixes graphql#150.
6dd3553
dahjelle added a commit to dahjelle/graphiql that referenced this issue Jul 28, 2016
@dahjelle
Fixes graphql#150.
c8aa3b7
dahjelle added a commit to dahjelle/graphiql that referenced this issue Jul 28, 2016
@dahjelle
Fixes graphql#150.
e55e4ac
dahjelle added a commit to dahjelle/graphiql that referenced this issue Aug 1, 2016
@dahjelle
Fixes graphql#150.
9cfb3a4
asiandrummer added a commit that referenced this issue Aug 1, 2016
@asiandrummer
Merge pull request #151 from dahjelle/fix-execute-button
d42b15a 
Fixes #150.
acao pushed a commit to acao/graphiql that referenced this issue Jun 1, 2019
@wincent
Merge pull request graphql#150 from graphql/greenkeeper/babel-plugin-…
5c20398 
…transform-es2015-computed-properties-6.24.1

Update babel-plugin-transform-es2015-computed-properties to the latest version 🚀
acao pushed a commit to acao/graphiql that referenced this issue Jun 5, 2019
@asiandrummer
cache type extensions hash, use it in getSchema (graphql#150)
b749b4c 
* cache type extensions hash, use it in getSchema
* fix tests and flow
acao pushed a commit that referenced this issue Feb 20, 2022
feat(automate release) (#150)

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
GPG key ID: 4AEE18F83AFDEB23
Expired
Verified
Learn about vigilant mode
a7df45f 
* feat(release): automate releases

via PAT
https://code.visualstudio.com/api/working-with-extensions/publishing-extension#get-a-personal-access-token
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@dahjelle