Bug 324171 - "Copy text" contextmenu entry doesn't work
Summary: "Copy text" contextmenu entry doesn't work
Status: RESOLVED FIXED
Alias: None
Product: dolphin
Classification: Applications
Component: bars: status (show other bugs)
Version: 4.11.0
Platform: unspecified Linux
: NOR minor
Target Milestone: ---
Assignee: Dolphin Bug Assignee
URL:
Keywords: junior-jobs
Depends on:
Blocks:
 
Reported: 2013-08-28 14:47 UTC by Kai Uwe Broulik
Modified: 2013-09-13 08:03 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 4.11.2


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Kai Uwe Broulik 2013-08-28 14:47:52 UTC
When right-clicking the status bar there is an entry "Copy text" which does nothing. I guess it's supposed to copy the statusbar text to the clipboard? It appears anywhere on the statusbar (eg. right-clicking the free space info results in the same menu)

Reproducible: Always

Steps to Reproduce:
1. Right click anywhere on the statusbar in Dolphin
2. Choose "Copy text"
3. Try to paste somewhere else
Actual Results:  
Nothing is pasted

Expected Results:  
The statusbar text(?) is pasted
Comment 1 Frank Reininghaus 2013-08-28 14:58:48 UTC
I never noticed this context menu entry. When looking at the code, it seems that text is only copied if it is not the "default text".

I cannot remember any use case in which this could be useful (in particular, since most error messages are not shown in the status bar any more).

The best solution is IMHO to remove this useless entry.
Comment 2 Emmanuel Pescosta 2013-08-28 17:27:24 UTC
> The best solution is IMHO to remove this useless entry.
+1
Comment 3 Frank Reininghaus 2013-09-10 08:24:01 UTC
Patch at https://git.reviewboard.kde.org/r/112355/
Comment 4 Frank Reininghaus 2013-09-12 21:34:18 UTC
Please use the "BUG" and "FIXED-IN" keywords when committing bug fixes in the future. Thanks.

https://projects.kde.org/projects/kde/applications/kde-baseapps/repository/revisions/4c17ce2c3abbdaf7d474ca8d604593e614159ce9
Comment 5 Kai Uwe Broulik 2013-09-13 08:03:22 UTC
Sorry, I just before pushed a fix to Plasma which didn't have a BKO entry and then forgot to add it here. Thanks for filling it in.