Bug 3903

Summary: Popups for new items in Web Admin has a confusing Delete button
Product: ThinLinc Reporter: Aaron Sowry <aaron>
Component: Web AdministrationAssignee: Pierre Ossman <ossman>
Status: NEW ---    
Severity: Normal CC: alexander.zeijlon, samuel
Priority: P2    
Version: 3.2.0   
Target Milestone: MediumPrio   
Hardware: PC   
OS: Unknown   
See Also: https://bugzilla.cendio.com/show_bug.cgi?id=8103
https://bugzilla.cendio.com/show_bug.cgi?id=8236
Acceptance Criteria:

Description Aaron Sowry cendio 2011-07-22 10:53:39 CEST
The behaviour when adding new locations/terminals in the GUI is inconsistent. For example, when adding a new Application Group in the TLDC module, you have the option to 'Delete' the yet-unsaved group. Clicking this has roughly the same effect as a 'Cancel' button, bringing you back to the list of currently saved groups. This same behaviour is exhibited when adding new Application Server Groups. So far, so good.

Doing the same thing for new locations/terminals however does not have this effect - the new group form remains displayed even after clicking delete.

A GUI-wide improvement may be to have a 'Cancel' button that can be used when editing a new group or making changes to an existing one.
Comment 1 Aaron Sowry cendio 2011-07-22 11:03:48 CEST
(In reply to comment #0)
> A GUI-wide improvement may be to have a 'Cancel' button that can be used when
> editing a new group or making changes to an existing one.
> 

Also, clicking "Delete" on an unsaved sub menu in TLDC gives the error message "No such menu". Doing the same thing with the "Yes, really delete..." checkbox checked results in the traceback described in bug #3904. A Cancel button would probably address this nicely.
Comment 2 Peter Åstrand cendio 2011-08-01 13:27:52 CEST
probably, the best solution is that the Delete button should not be visible at all if editing a new object. 
Comment 3 Frida Flodin cendio 2023-03-20 09:11:38 CET
*** Bug 8090 has been marked as a duplicate of this bug. ***
Comment 4 Alexander Zeijlon cendio 2023-04-05 08:52:41 CEST
There is currently a special case for not showing the delete button in TLDC⇾Menu structure that could be moved to our tllib function for save and delete buttons such that it can be used elsewhere.