[Uml-devel] kdesdk/umbrello/umbrello

[Sebastian Stein]

CVS commit by sstein: 

fix bug #65635:
There is a timer in UMLApp, which forces every 500ms a check (UMLApp::slotCopyChanged()) if the state of copy/paste is correctly. Now, whenever a change of selection happens, we call this check manually as well.

Another solution would be to have a signal signalSelectionChanged() in umlview and umlwidget and connect it to UMLApp::slotCopyChanged(). I think this is a little bit overkill, but of course we can change it to this way...


  M +4 -0      umlview.cpp   1.113
  M +5 -0      umlwidget.cpp   1.64


--- kdesdk/umbrello/umbrello/umlwidget.cpp  #1.63:1.64
@@ -786,4 +787,8 @@ void UMLWidget::setSelected(bool _select
         m_bSelected = _select;
         update();
+
+        /* selection changed, we have to make sure the copy and paste items
+         * are correctly enabled/disabled */
+        UMLApp::app()->slotCopyChanged();
 }
 

--- kdesdk/umbrello/umbrello/umlview.cpp  #1.112:1.113
@@ -968,4 +968,8 @@ void UMLView::setSelected(UMLWidget * w,
         if( count == 2 )
                 updateDocumentation( true );//clear doc window
+
+        /* selection changed, we have to make sure the copy and paste items
+         * are correctly enabled/disabled */
+        UMLApp::app()->slotCopyChanged();
 }
 ////////////////////////////////////////////////////////////////////////////////////////////////////