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

Close mutation editor on submission #835

Closed
4 tasks done
wxtim opened this issue Nov 4, 2021 · 7 comments · Fixed by #1108, #1160 or #1428
Closed
4 tasks done

Close mutation editor on submission #835

wxtim opened this issue Nov 4, 2021 · 7 comments · Fixed by #1108, #1160 or #1428
Assignees
Labels
good first issue Good for newcomers UX/UI User experience and interface improvements
Milestone

Comments

@wxtim
Copy link
Member

wxtim commented Nov 4, 2021

Describe exactly what you would like to see in an upcoming release

[edit] updated, see comments bellow

[/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.

image

Pull requests welcome!

@wxtim wxtim added accessibility enhancement good first issue Good for newcomers question Flag this as a question for the next Cylc project meeting. labels Nov 4, 2021
@wxtim wxtim modified the milestones: 2.0, 1.0 Nov 4, 2021
@hjoliver
Copy link
Member

hjoliver commented Nov 4, 2021

I agree. "Cancel" could easily be confused with trying to cancel the mutation you've just submitted.

@oliver-sanders
Copy link
Member

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:

  • Remove the old task icon status thinggy (was never intended to be released).
  • Make the dialogue box close on successful submission of a mutation.
  • Keep the dialogue box open for an unsuccessful submission and raise an Alert (if not already done).

@MetRonnie MetRonnie changed the title Close "windows" with ❌ Close mutation form "windows" with ❌ Nov 9, 2021
@oliver-sanders oliver-sanders changed the title Close mutation form "windows" with ❌ Close mutation editor on submission Nov 10, 2021
@oliver-sanders
Copy link
Member

Two likes no screams, removing the question label.

@oliver-sanders oliver-sanders removed the question Flag this as a question for the next Cylc project meeting. label Nov 29, 2021
@MetRonnie MetRonnie modified the milestones: 1.0, 1.x Feb 4, 2022
@MetRonnie
Copy link
Member

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

@oliver-sanders
Copy link
Member

(alternatively ensure the cancel button is always visible e.g. via a sticky footer)

@oliver-sanders oliver-sanders modified the milestones: 1.x, Pending Jun 8, 2022
@oliver-sanders
Copy link
Member

oliver-sanders commented Oct 17, 2022

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).

@oliver-sanders oliver-sanders added the UX/UI User experience and interface improvements label Oct 17, 2022
@oliver-sanders
Copy link
Member

oliver-sanders commented Dec 1, 2022

Note this will likely be closed by #1144

They did for edit-runtime dialog. I need to check they do for others/

@wxtim wxtim mentioned this issue Dec 5, 2022
6 tasks
This was linked to pull requests Dec 15, 2022
@oliver-sanders oliver-sanders removed this from the Pending milestone Aug 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers UX/UI User experience and interface improvements
Projects
None yet
5 participants