bug #232: fit all on image show

[modules/hydro.git] / src / HYDROGUI / HYDROGUI_ShowHideOp.cxx

diff --git a/src/HYDROGUI/HYDROGUI_ShowHideOp.cxx b/src/HYDROGUI/HYDROGUI_ShowHideOp.cxx
index d06c59f26ce4c267f84d89fd05db87ba0c3d6e84..6ff798943be9ba6eb0bf539e0169ac2ac717d6f2 100644 (file)
--- a/src/HYDROGUI/HYDROGUI_ShowHideOp.cxx
+++ b/src/HYDROGUI/HYDROGUI_ShowHideOp.cxx
@@ -82,19 +82,39 @@ void HYDROGUI_ShowHideOp::startOperation()
 
   int anUpdateFlags = 0;
   SUIT_ViewManager* aVTKMgr = 0;
+  SUIT_ViewManager* aViewMgr = module()->getApp()->activeViewManager();
+  if ( aViewMgr )
+  {
+    if ( aViewMgr->getType() == GraphicsView_Viewer::Type() )
+    {
+      anUpdateFlags |= UF_Viewer;
+    }
+    else if ( aViewMgr->getType() == OCCViewer_Viewer::Type() )
+    {
+      anUpdateFlags |= UF_OCCViewer;
+    }
+    else if ( aViewMgr->getType() == SVTK_Viewer::Type() )
+    {
+      anUpdateFlags |= UF_VTKViewer;
+    }
+  }
 
+  bool isFoundImage = false;
   // for selected objects
   if( myId == ShowId || myId == ShowOnlyId || myId == HideId )
   {
     HYDROData_SequenceOfObjects aSeq = HYDROGUI_Tool::GetSelectedObjects( module() );
   
-    bool aVisibility = myId == ShowId || myId == ShowOnlyId;
+    bool aVisibility = myId == ShowId || myId == ShowOnlyId || myId == ShowAllId;
     Handle( HYDROData_Entity ) anObject;
     for( Standard_Integer anIndex = 1, aLength = aSeq.Length(); anIndex <= aLength; anIndex++ )
     {
       anObject = aSeq.Value( anIndex );
       if( !anObject.IsNull() )
       {
+        if ( anObject->GetKind() == KIND_IMAGE )
+          isFoundImage = true;
+
         module()->setObjectVisible( aViewId, anObject, aVisibility );
         if ( anObject->GetKind() == KIND_REGION )
         {
@@ -114,15 +134,18 @@ void HYDROGUI_ShowHideOp::startOperation()
         }
         else if ( anObject->GetKind() == KIND_BATHYMETRY && aVisibility )
         {
-          // Activate VTK viewer if show a bathymetry
-          aVTKMgr = module()->getApp()->viewManager( SVTK_Viewer::Type() );
-          if ( !aVTKMgr )
+          if ( !(anUpdateFlags & UF_VTKViewer) )
           {
-            aVTKMgr = module()->getApp()->createViewManager( SVTK_Viewer::Type() );
-          }
-          if ( aVTKMgr )
-          {
-            module()->setObjectVisible( (size_t)aVTKMgr->getViewModel(), anObject, aVisibility );
+            // Activate VTK viewer if show a bathymetry
+            aVTKMgr = module()->getApp()->viewManager( SVTK_Viewer::Type() );
+            if ( !aVTKMgr )
+            {
+              aVTKMgr = module()->getApp()->createViewManager( SVTK_Viewer::Type() );
+            }
+            if ( aVTKMgr )
+            {
+              module()->setObjectVisible( (size_t)aVTKMgr->getViewModel(), anObject, aVisibility );
+            }
           }
         }
       }
@@ -131,7 +154,8 @@ void HYDROGUI_ShowHideOp::startOperation()
 
   if ( myId == ShowOnlyId || myId == ShowId || myId == ShowAllId )
   {
-    anUpdateFlags |= UF_FitAll;
+    if( isFoundImage || myId == ShowAllId )
+      anUpdateFlags |= UF_FitAll;
   }
 
   // Set VTK viewer active if show a bathymetry
@@ -140,22 +164,6 @@ void HYDROGUI_ShowHideOp::startOperation()
     anUpdateFlags |= UF_VTKViewer;
     aVTKMgr->setShown( true );
   }
-  SUIT_ViewManager* aViewMgr = module()->getApp()->activeViewManager();
-  if ( aViewMgr )
-  {
-    if ( aViewMgr->getType() == GraphicsView_Viewer::Type() )
-    {
-      anUpdateFlags |= UF_Viewer;
-    }
-    else if ( aViewMgr->getType() == OCCViewer_Viewer::Type() )
-    {
-      anUpdateFlags |= UF_OCCViewer;
-    }
-    else if ( aViewMgr->getType() == SVTK_Viewer::Type() )
-    {
-      anUpdateFlags |= UF_VTKViewer;
-    }
-  }
 
   module()->update( anUpdateFlags );
   commit();