-
Notifications
You must be signed in to change notification settings - Fork 27
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
Close mutation editor on submission #835
Comments
I agree. "Cancel" could easily be confused with trying to cancel the mutation you've just submitted. |
The box should close once the mutation has been submitted. At the moment it doesn't for debug purposes, I think when you submit a mutation you still get a task icon showing the status of the mutation (i.e. waiting, submitted or submit-failed). Propose the following:
|
Two likes no screams, removing the question label. |
The mutation menu could also do with an "x" close button, because for small viewports there is likely to barely be any room to tap outside the menu in order to close it. I have added this (and the close button for the mutation editor form) as a bullet point |
(alternatively ensure the cancel button is always visible e.g. via a sticky footer) |
Suggest putting the submit and cancel buttons in a sticky footer at the bottom of the form so they are always visible as this will also be useful for the submit button (we don't always need to scroll to the bottom). |
They did for edit-runtime dialog. I need to check they do for others/ |
Describe exactly what you would like to see in an upcoming release
[edit] updated, see comments bellow
Add a close "x" button to the top right of the mutation editor form, as well as the mutation menuPin the action button ("submit", "cancel" etc) to the bottom of the form so they're visible at all times (Feature mutation form sticky buttons #1428)[/edit]
The following dialog is fully functional but some users may find having to close with the cancel button confusing.
In my opinion these dialogs should have a cross which copies the functionality of the cancel button at either the top right or top left corner.
Pull requests welcome!
The text was updated successfully, but these errors were encountered: