Bug 284531

Summary: Standard Copy (CTRL-C) shortcut is not operational in chat windows
Product: [Unmaintained] telepathy Reporter: Luca Beltrame <lbeltrame>
Component: text-uiAssignee: Telepathy Bugs <kde-telepathy-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: alinm.elena, kde
Priority: NOR    
Version: git-latest   
Target Milestone: 0.4.0   
Platform: openSUSE   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Luca Beltrame 2011-10-20 09:43:39 UTC
Version:           git-latest (using Devel) 
OS:                Linux

In telepathy-text-ui, there is no shortcut to copy text (CTRL-C). It is still possible to copy text through the right click menu, or using X11's clipboard.

Reproducible: Always

Steps to Reproduce:
1. Open a chat window
2. Input text
3. Highlight text
4. Press CTRL-C
5. Paste the text

Actual Results:  
The selected text is not pasted

Expected Results:  
The selected text is pasted

Git master version from yesterday.
Comment 1 Alin M Elena 2011-10-20 09:55:19 UTC
ctrl+c does the job... at least part of it...
if you check the clipboard the text is there...
it is just ctrl+v that pastes you the wrong one...
Comment 2 David Edmundson 2011-10-20 11:18:54 UTC
This is a valid bug, patch is already in reviewboard.

https://git.reviewboard.kde.org/r/102908/ 

If no-one reviews it within another 2 days I'm just going to ship it.
Comment 3 David Edmundson 2011-10-20 23:35:18 UTC
Git commit e8df9c842805dd449704f85cb8e8ab23c6341d5b by David Edmundson.
Committed on 18/10/2011 at 21:20.
Pushed by davidedmundson into branch 'master'.

Tidy up all key handling code.

Remove signals and slots, and use event propogation where it makes more sense to pass events to parents.

REVIEW: 102908
BUG: 277231
BUG: 284531

M  +0    -5    lib/adium-theme-view.cpp
M  +0    -1    lib/adium-theme-view.h
M  +30   -6    lib/chat-text-edit.cpp
M  +2    -3    lib/chat-text-edit.h
M  +14   -9    lib/chat-widget.cpp
M  +0    -1    lib/chat-widget.h

http://commits.kde.org/telepathy-text-ui/e8df9c842805dd449704f85cb8e8ab23c6341d5b