Project

General

Profile

Bug #11908

Unable to cancel "request is being processed" window when editing a host

Added by Julien Pivotto almost 4 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Web Interface
Target version:
Difficulty:
Triaged:
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

when you click "edit" in a host you have that modal:

"Loading... Spinner
Please wait while your request is being processed"

The modal has a close (X) button, I was expecting it just to cancel the action but it only hides the modal and the edit request keeps processing.

Should we:

1. Remove the Close button
2. Really cancel the action

I think this applies in several locations


Related issues

Related to Foreman - Bug #2592: No indication when waiting for host dashboard button action to completeClosed2013-05-30

Associated revisions

Revision 2b647d93 (diff)
Added by Alon Goldboim almost 4 years ago

Fixes #11908 - removed the close button from the loading alert

Revision 53b9424e (diff)
Added by Alon Goldboim almost 4 years ago

Fixes #11908 - removed the close button from the loading alert

(cherry picked from commit 2b647d937e739b7e87b371d971a8e2ebdf65fa63)

History

#1 Updated by Julien Pivotto almost 4 years ago

  • Subject changed from Unable to cancel ah to Unable to cancel editing a host

#2 Updated by Julien Pivotto almost 4 years ago

  • Description updated (diff)

#3 Updated by Dominic Cleal almost 4 years ago

  • Related to Bug #2592: No indication when waiting for host dashboard button action to complete added

#4 Updated by Dominic Cleal almost 4 years ago

  • Subject changed from Unable to cancel editing a host to Unable to cancel "request is being processed" window when editing a host
  • Category set to Web Interface

#5 Updated by Alon Goldboim almost 4 years ago

  • Assignee set to Alon Goldboim

#6 Updated by Julien Pivotto almost 4 years ago

Alon, I am not working on this now, so feel free to take it! :)

#7 Updated by Alon Goldboim almost 4 years ago

Julien, great, on it.

#8 Updated by Alon Goldboim almost 4 years ago

Its bad UX but i can see that the purpose of the close button was to close the alert panel and not cancel the routing.
Im removing the btn.

#9 Updated by The Foreman Bot almost 4 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/2817 added
  • Pull request deleted ()

#10 Updated by Julien Pivotto almost 4 years ago

This is annoying when you have a request that fails. It would be better to really be able to cancel the request (maybe with an explicit cancel button)

Another behaviour would be that when a request fails an error message replaces modal.

It is very annoying not to have the close button when you have 500 errors or network errors.

#11 Updated by Dominic Cleal almost 4 years ago

  • Legacy Backlogs Release (now unused) set to 63

Seems your comment was missed, it's been merged without cancelling the request - only removing the button.

#12 Updated by Alon Goldboim almost 4 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#13 Updated by Julien Pivotto almost 4 years ago

:(

Also available in: Atom PDF