Bug 50984 - KWord: wrong context menu for pictures
Summary: KWord: wrong context menu for pictures
Status: CLOSED FIXED
Alias: None
Product: kword
Classification: Miscellaneous
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Thomas Zander
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-11-20 21:52 UTC by Nicolas Goutte
Modified: 2003-02-19 23:23 UTC (History)
0 users

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 Nicolas Goutte 2002-11-20 21:52:46 UTC
Version:            (using KDE Devel)
Installed from:    Compiled sources
Compiler:          gcc 3.2 
OS:          Linux

While cliking the RMB on an inlined picture, I get the normal (text) context menu and not the one for picture.

Therefore you cannot change or save the picture.

Have a nice day/evening/night!
Comment 1 Nicolas Goutte 2003-01-12 20:14:19 UTC
I have found additional information on this bug: 
in KWView::openPopupMenuEditFrame there is a check that goes wrong. 
 
A variable nbFrame is defined like that 
int nbFrame=m_doc->getSelectedFrames().count(); 
and if nbFrame==1, the correct RMB menu is generated. 
 
But for inline picture nbFrame==0, so the RMB menu is not genereated correctly. 
 
Is it normal that no frame exist or not? If it is normal, the fix should be easily. 
 
Have a nice day/evening/night! 
Comment 2 Clarence Dang 2003-02-19 09:42:20 UTC
Subject: koffice/kword

CVS commit by dang: 


should fix bug where you can't select an inline frame (nor right-click on it)
CCMAIL: 50984-done@bugs.kde.org


  M +1 -1      kwcanvas.cc   1.429


--- koffice/kword/kwcanvas.cc  #1.428:1.429
@@ -525,5 +525,5 @@ void KWCanvas::contentsMousePressEvent( 
         //kdDebug() << "contentsMousePressEvent meaning=" << m_mouseMeaning << endl;
         Q_ASSERT( m_mouseMeaning < MEANING_TOPLEFT ); // during resizing, the resizehandles are supposed to get the events
-        if ( m_mouseMeaning == MEANING_MOUSE_MOVE )
+        if ( m_mouseMeaning == MEANING_MOUSE_MOVE || m_mouseMeaning == MEANING_MOUSE_SELECT )
         {
             if ( m_currentFrameSetEdit )