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

Do not show dropdown for execute button if an operation is currently running #2885

Merged
merged 1 commit into from Nov 10, 2022

Conversation

simhnna
Copy link
Contributor

@simhnna simhnna commented Nov 10, 2022

If you have multiple operations defined in the editor, clicking the execute button will show a dropdown to choose which operation to run.

If an operation is running, the stop button still shows the dropdown. Since only one operation can be running at any point in time, showing the dropdown is useless. This merge request makes sure that the dropdown is only shown, if no operation is currently running

@changeset-bot
Copy link

changeset-bot bot commented Nov 10, 2022

🦋 Changeset detected

Latest commit: f889d7d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
graphiql Patch
@graphiql/react Patch
@graphiql/plugin-explorer Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@simhnna simhnna force-pushed the do-not-show-dropdown-for-stop branch from 8ee94be to f889d7d Compare November 10, 2022 17:20
'@graphiql/react': patch
---

Fix stop execution button showing a dropdown
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Fix stop execution button showing a dropdown
Bugfix: Stop execution button from showing a dropdown during operation fetch

Copy link
Member

@acao acao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me! thanks for this PR @simhnna

@acao acao merged commit 8f92648 into graphql:main Nov 10, 2022
@acao acao mentioned this pull request Nov 10, 2022
@simhnna simhnna deleted the do-not-show-dropdown-for-stop branch November 12, 2022 23:04
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.

None yet

2 participants