Bug 259624 - context menu in mail reader missing, e.g. to save multiple attachment
Summary: context menu in mail reader missing, e.g. to save multiple attachment
Status: RESOLVED WORKSFORME
Alias: None
Product: kmail2
Classification: Applications
Component: general (show other bugs)
Version: 2.0.89
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords: investigated, triaged
Depends on:
Blocks:
 
Reported: 2010-12-12 17:53 UTC by m.wege
Modified: 2018-10-27 02:27 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 m.wege 2010-12-12 17:53:09 UTC
Version:           2.0.89 (using KDE 4.5.85) 
OS:                Linux

The context menu in mail reader seems to be missing when I click on the text of a mail. If I mark text, there is no context menu. I also remember there was an option to save to save multiple attachment from on email at once.

Reproducible: Always




OS: Linux (i686) release 2.6.37-7-generic-pae
Compiler: cc
Comment 1 m.wege 2010-12-17 23:41:34 UTC
There is also missing the context menu option to show the message source.
Comment 2 Torgny Nyblom 2010-12-26 21:27:43 UTC
commit 4820015984b4740fa14b45a0717057cae1472b45
branch master
Author: Torgny Nyblom <kde@nyblom.org>
Date:   Sun Dec 26 21:20:01 2010 +0100

    Reenable context menu for empty links, ie right click on message.
    
    Might affect kmail mobile as described in bugs: 249620, 249905
    
    CCBUG: 259624
    CCMAIL: adam@kde.org

diff --git a/messageviewer/mailwebview_webkit.cpp b/messageviewer/mailwebview_webkit.cpp
index 03b81ce..2d5c24b 100644
--- a/messageviewer/mailwebview_webkit.cpp
+++ b/messageviewer/mailwebview_webkit.cpp
@@ -70,8 +70,10 @@ bool MailWebView::event( QEvent *event )
     const QWebFrame * const frame = page()->currentFrame();
     const QWebHitTestResult hit = frame->hitTestContent( contextMenuEvent->pos() );
     kDebug() << "Right-clicked URL:" << hit.linkUrl();
+#ifdef Q_OS_WINCE
     if ( !hit.linkUrl().isEmpty() )
-      emit popupMenu( hit.linkUrl().toString(), mapToGlobal( contextMenuEvent->pos() ) );
+#endif
+    emit popupMenu( hit.linkUrl().toString(), mapToGlobal( contextMenuEvent->pos() ) );
     event->accept();
     return true;
   }
Comment 3 Tobias Koenig 2011-06-04 13:06:15 UTC
Does it work now?
Comment 4 m.wege 2011-06-04 14:40:49 UTC
Can not tell, there have been not packages for Kubuntu since Beta2, so I have reverted to Kmail1.
Comment 5 Andrew Crouthamel 2018-09-22 01:57:45 UTC
Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least 15 days. Please provide the requested information as soon as possible and set the bug status as REPORTED. Due to regular bug tracker maintenance, if the bug is still in NEEDSINFO status with no change in 30 days, the bug will be closed as RESOLVED > WORKSFORME due to lack of needed information.

For more information about our bug triaging procedures please read the wiki located here: https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

If you have already provided the requested information, please set the bug status as REPORTED so that the KDE team knows that the bug is ready to be confirmed.

Thank you for helping us make KDE software even better for everyone!
Comment 6 Andrew Crouthamel 2018-10-27 02:27:52 UTC
Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least 30 days. The bug is now closed as RESOLVED > WORKSFORME due to lack of needed information.

For more information about our bug triaging procedures please read the wiki located here: https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

Thank you for helping us make KDE software even better for everyone!