[prev in list] [next in list] [prev in thread] [next in thread] 

List:       koffice-devel
Subject:    kspread: deleting chart
From:       Ariya Hidayat <ariya () kde ! org>
Date:       2003-11-15 4:20:34
[Download RAW message or body]

This is the problem describe in bug #37152. Actually chart (and other 
embedded part) can be deleted without problem, just click once to select 
it and then right-click on the frame border to pop-up a menu for 
deleting the embedded document.

The confusing thing is that the frame borders aren't displayed properly 
when the chart is clicked by user. The patch fixes this problem. A thick 
frame will appear to indicate that an embedded part (including chart) is 
selected. Of course, grips for part resizing are also there.

I will commit when nobody objects.

Best regards,

Ariya Hidayat
http://ariya.pandu.org

["deletechart.patch" (text/plain)]

--- kspread_view.cc.orig	2003-11-07 09:55:37.000000000 +0100
+++ kspread_view.cc	2003-11-15 04:56:21.000000000 +0100
@@ -6426,10 +6436,17 @@ void KSpreadView::slotChildSelected( KoD
         m_transformToolBox->setDocumentChild( ch );
     }
   }
+
+  m_pDoc->emitBeginOperation( false );
+  m_pTable->setRegionPaintDirty(QRect(QPoint(0,0), QPoint(KS_colMax, KS_rowMax)));
+  m_pDoc->emitEndOperation();
+  paintUpdates();
 }
 
 void KSpreadView::slotChildUnselected( KoDocumentChild* )
 {
+  if ( m_pTable && !m_pTable->isProtected() )
+  {
     m_transform->setEnabled( FALSE );
 
     if ( !m_transformToolBox.isNull() )
@@ -6437,6 +6454,12 @@ void KSpreadView::slotChildUnselected( K
         m_transformToolBox->setEnabled( FALSE );
     }
     deleteEditor( true );
+  }
+
+  m_pDoc->emitBeginOperation( false );
+  m_pTable->setRegionPaintDirty(QRect(QPoint(0,0), QPoint(KS_colMax, KS_rowMax)));
+  m_pDoc->emitEndOperation();
+  paintUpdates();
 }
 
 


_______________________________________________
koffice-devel mailing list
koffice-devel@mail.kde.org
https://mail.kde.org/mailman/listinfo/koffice-devel


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic