[Uml-devel] kdesdk/umbrello/umbrello

Sebastian Stein seb.kde at hpfsc.de
Wed Feb 4 04:43:07 UTC 2004


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();
 }
 






More information about the umbrello-devel mailing list