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

List:       kde-commits
Subject:    kdesdk/umbrello/umbrello
From:       Sebastian Stein <seb.kde () hpfsc ! de>
Date:       2004-02-04 12:40:45
Message-ID: 20040204124045.324B99914 () office ! kde ! org
[Download RAW message or body]

CVS commit by sstein: 

I moved the setModified to removeWidget() and removeAssoc() directly. The problem \
with this approach is, that you have to click several times undo, after removing a \
class and some associations. On some undo clicks the user won't see any differences. \
Please have a look at this. I'm not sure, if I should backport this 3.2.


  M +2 -1      umlview.cpp   1.122


--- kdesdk/umbrello/umbrello/umlview.cpp  #1.121:1.122
@@ -900,4 +900,5 @@ void UMLView::removeWidget(UMLWidget * o
                 m_WidgetList.remove(o);
         delete o;
+        m_pDoc->setModified();
 }
 ////////////////////////////////////////////////////////////////////////////////////////////////////
 @@ -1175,5 +1176,4 @@ void UMLView::deleteSelection()
         //make sure list empty - it should be anyway, just a check.
         m_SelectedList.clear();
-        m_pDoc->setModified();
 }
 
@@ -1999,4 +1999,5 @@ void UMLView::removeAssoc(AssociationWid
 //      pAssoc->cleanup();
         m_AssociationList.remove(pAssoc); // will delete our association
+        m_pDoc->setModified();
 }
 


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