Version: (using KDE 4.3.0)
Installed from: Fedora RPMs
The word "Copy" is misleading when there is no text highlighted on the konsole. I think the menu option should be renamed to "Clear Clipboard" or something similar, or else greyed out (which many other user interfaces do, such as this textarea element in Mozilla Firefox.)
I would prefer "Clear Clipboard" because that is what it actually does, and greying it out prevents me from clearing my clipboard with a right click which I think would be innovative and cool.
Not to mention easy to implement (hopefully) ;-)
Okay, so what do you think about this idea?
I don't agree to change the label because it could add extra confusion to users: nothing is selected, so nothing is copied. If having the copy action which copies nothing causes difficulties, it should be disabled, not renamed.
Yes, please disable it at a minimum.
The renaming option was only if you wanted to be innovative.
But the status quo would be to just disable it. It is quite confusing as it is now, otherwise I would not have filed this bug.
This is coming from an experienced computer programmer and 10-year veteran linux user. I'm sure if it confuses me, it will confuse my Aunt Tilly. ;-)
BTW, I became confused when I was trying to select a URL link in the konsole using the context menu. I had known that you could copy links from the konsole using the context menu, so I activated the context menu with a right click on top of the URL and noticing the Copy option was available, I selected that. Then I had to figure out why it wasn't actually copying the link. After closer inspection I noticed the "Copy Link Location" menu option in the context menu. So this is actually where the confusion lied with me. Had the copy option been greyed out, I'm sure I would have selected "Copy Link Location" on the first try.
I renamed this wish per #2 and so this wish could be tracked.
Git commit f5772ab06b4f640b5cdb3d5be8f93183f2258cd8 by Jekyll Wu.
Committed on 26/08/2011 at 18:56.
Pushed by jekyllwu into branch 'master'.
Enable copy action only when some text are selected.
M +12 -0 src/Session.h
M +11 -0 src/SessionController.cpp
M +9 -0 src/Emulation.h
M +9 -0 src/Emulation.cpp
M +7 -0 src/SessionController.h
M +2 -0 src/ViewManager.cpp
M +7 -0 src/Session.cpp