Bug 148509 - message plain text viewer: KDE shortcuts does not work, only Qt basic
Summary: message plain text viewer: KDE shortcuts does not work, only Qt basic
Status: RESOLVED FIXED
Alias: None
Product: kmail
Classification: Applications
Component: messageviewer (show other bugs)
Version: 1.9.7
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords: triaged
Depends on:
Blocks:
 
Reported: 2007-08-04 17:47 UTC by Maciej Pilichowski
Modified: 2009-08-01 23:33 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Maciej Pilichowski 2007-08-04 17:47:29 UTC
Version:            (using KDE KDE 3.5.7)
Installed from:    SuSE RPMs

Compare for example ctrl+c and ctrl+insert.

(I am seeing this bug from time to time in various apps and I am posting separate reports for such bugs, I hope it is ok -- i.e. I should not post one "summarized" report).
Comment 1 Bram Schoenmakers 2007-08-05 11:02:04 UTC
I don't understand, please supply more information.
Comment 2 Maciej Pilichowski 2007-08-05 14:41:53 UTC
Ctrl+insert is a shortcut for copy. Ctrl+C is a shortcut for copy. Both are KDE shortcuts.

KMail message plain text viewer supports only the latter one, i.e. the first one does not work. And it should. Bug.

My wild guess is text viewer uses regular Qt widgets, not KDE -- and I saw such implementation in various KDE apps with the same result ==> only the first shortcut is supported, not the alternative.
Comment 3 Philip Rodrigues 2007-08-05 19:32:00 UTC
These problems are now found by the "krazy" source code checker at www.englishbreakfastnetwork.org, so I don't think it's necessary to file bug reports for them. In fact, they're probably fairly easy to fix - you might like to give it a go.
Comment 4 Maciej Pilichowski 2007-08-05 20:33:06 UTC
Thank you Philip, so I do not post reports against this kind of bugs. Btw. I've check the results of verifying KDE code, it is really great tool.
Comment 5 Thomas McGuire 2007-08-06 20:37:46 UTC
>These problems are now found by the "krazy" source code checker
Can you give a link to the specific page which lists this problem, please?


See also bug 147794.
Comment 6 Philip Rodrigues 2007-08-25 13:47:20 UTC
There are some other notes about this on bug 148159

For the krazy output for this issue, see item 19 on:
http://www.englishbreakfastnetwork.org/krazy/reports/kde-4.0/kdepim/kmail/index.html
Comment 7 Michael Leupold 2009-04-05 11:51:55 UTC
This problem seems to be fixed now. Ctrl+Ins works fine in the plain text view in KMail 1.11.0.
Comment 8 Christophe Marin 2009-08-01 23:33:55 UTC
Can't reproduce anymore. Closing.