Page MenuHomePhabricator

[Segmentation] Pressing DEL should delete a label
Open, WishlistPublic

Description

As suggested in a tested checklist, being able to use the keyboard instead of the context-menu for deleting selected labels should be implemented.

Event Timeline

kalali triaged this task as Normal priority.Mar 28 2022, 4:27 PM
kalali created this task.
kislinsk lowered the priority of this task from Normal to Wishlist.Mar 28 2022, 8:23 PM
kislinsk added a subscriber: kislinsk.

Same for F2 regarding renaming of labels. I would also say double-click, but as it turned out, double-click is already used to jump to the center of the label.

I was not able to react on these key events with event filters like it is done in the Data Manager as they seem to be swallowed by some other widget. Not gonna say it cannot be done but it seems like this task is tricky and potentially time consuming.

I think with the new widget it should be easy. What are the keys actions wanted now?

Still does not work for Del. For a sequence like CTRL+L CTRL+D it works.

Do we have somewhere a list (or better user documentation) of all shortcutes so far?

May be we should revise and refine it (together with the naming of action) to make it easie to remember and use the shortcuts.

E.g. Erease Label -> Cleare label to make it easier to differentiate from Remove Label
E.g. hotkey L+A for add (which is also the hint of the button) instead L+N. Then L+N could be used for reName and in turn L+R could be used for Remove Label

Unknown Object (User) added a subtask: Unknown Object (Maniphest Task).Jun 28 2023, 9:32 AM
Unknown Object (User) added a subscriber: Unknown Object (User).Jun 28 2023, 9:51 AM
This comment was removed by kislinsk.