]> SALOME platform Git repositories - modules/shaper.git/commitdiff
Salome HOME
Issue #1527:Do not disable remove button on input wrong name
authorvsv <vitaly.smetannikov@opencascade.com>
Fri, 27 May 2016 14:53:02 +0000 (17:53 +0300)
committervsv <vitaly.smetannikov@opencascade.com>
Fri, 27 May 2016 14:53:02 +0000 (17:53 +0300)
src/ParametersPlugin/ParametersPlugin_WidgetParamsMgr.cpp

index a340cdb3e6e9fc1bec3824cb465811c1c463a237..81f851319ca3533cfe26b1d70f5eebbce695ebe6 100644 (file)
@@ -417,7 +417,7 @@ void ParametersPlugin_WidgetParamsMgr::onCloseEditor(QWidget* theEditor,
           aText.replace(" ", "");
         }
         if (hasName(aText)) {
-          myMessage = tr("Name %1 already exists.").arg(aText);
+          myMessage = tr("Name '%1' already exists.").arg(aText);
           QTimer::singleShot(50, this, SLOT(sendWarning()));
           return;
         }
@@ -457,6 +457,8 @@ void ParametersPlugin_WidgetParamsMgr::onCloseEditor(QWidget* theEditor,
   if (aColumn == Col_Equation)
     updateParametersPart();
   updateFeaturesPart();
+          
+  onSelectionChanged();
 }
 
 void ParametersPlugin_WidgetParamsMgr::updateItem(QTreeWidgetItem* theItem, 
@@ -700,9 +702,9 @@ void ParametersPlugin_WidgetParamsMgr::sendWarning()
 
 void ParametersPlugin_WidgetParamsMgr::onSelectionChanged()
 {
+  QList<QTreeWidgetItem*> aItemsList = myTable->selectedItems();
   bool isValid = checkIsValid();
   if (isValid) {
-    QList<QTreeWidgetItem*> aItemsList = myTable->selectedItems();
     bool isParameter = false;
     foreach(QTreeWidgetItem* aItem, aItemsList) {
       if (aItem->parent() == myParameters) {
@@ -711,15 +713,16 @@ void ParametersPlugin_WidgetParamsMgr::onSelectionChanged()
       }
     }
     myInsertBtn->setEnabled(isParameter);
-    myRemoveBtn->setEnabled(isParameter);
+    //myRemoveBtn->setEnabled(isParameter);
     myUpBtn->setEnabled(isParameter);
     myDownBtn->setEnabled(isParameter);
   } else {
     myInsertBtn->setEnabled(false);
-    myRemoveBtn->setEnabled(false);
+    //myRemoveBtn->setEnabled(false);
     myUpBtn->setEnabled(false);
     myDownBtn->setEnabled(false);
   }
+  myRemoveBtn->setEnabled(!aItemsList.isEmpty());
 }
 
 void ParametersPlugin_WidgetParamsMgr::enableButtons(bool theEnable)