Add Copy Command Label to Keyboard Shortcuts.#125563
Conversation
1d049c3 to
2e5bf7e
Compare
|
@sandy081 did you get time to take a l:eyes:k at this yet? |
|
Instead of context menu can you please make the text selectable? |
It is a good idea! What way should it be selectable? The UI already allows:
...for this reason, I am not sure what is a good UX here. What do you think @sandy081 🤔 ? Also, we have 2 ways of copying aspects of the command via context menu already. Is it orthogonal to have a 3rd way of copying something from the command somewhere else? Alternatively, what do you think of adding a copy button? I am not sure it belongs in the action buttons at the beginning of the row, but could go inside the |
|
Please ignore my previous comment. After thinking more I felt initial approach of providing the copy command in the context menu makes sense. Please let me know if the changes with context menu option are ready for review. |
|
@sandy081 it is ready for review 👍 |


This PR fixes #125092.