Bug 284531

Summary:	Standard Copy (CTRL-C) shortcut is not operational in chat windows
Product:	[Frameworks and Libraries] telepathy	Reporter:	Luca Beltrame <lbeltrame>
Component:	text-ui	Assignee:	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:

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