Task #12831 (closed)
Opened 9 years ago
Closed 9 years ago
Web handle 'DEL' key is jsTree
| Reported by: | wmoore | Owned by: | wmoore |
|---|---|---|---|
| Priority: | major | Milestone: | Unscheduled |
| Component: | Web | Version: | n.a. |
| Keywords: | n.a. | Cc: | |
| Resources: | n.a. | Referenced By: | n.a. |
| References: | n.a. | Remaining Time: | n.a. |
| Sprint: | n.a. |
Description
The default behaviour for Delete key is not disabled for jsTree, so this ends up removing nodes but is not handled anywhere so the delete doesn't happen.
Need to catch the node removal event and pop-up the confirm dialog and handle delete accordingly.
This might be similar to this line in https://github.com/openmicroscopy/openmicroscopy/blob/develop/components/tools/OmeroWeb/omeroweb/webclient/templates/webclient/data/containers.html#L927
.bind("move_node.jstree", function (e, data) {
but use "remove_node" and call OME.handleDelete(). And if the user clicked "Cancel" instead of confirm then we'd roll back.
Alternatively, simply disable the Delete key.
Change History (1)
comment:1 Changed 9 years ago by wmoore
- Resolution set to fixed
- Status changed from new to closed
Note: See
TracTickets for help on using
tickets.
You may also have a look at Agilo extensions to the ticket.
Fixed in https://github.com/openmicroscopy/openmicroscopy/pull/3771