Few string fixes from Dan.

[lyx.git] / src / frontends / qt / InsetParamsDialog.cpp

diff --git a/src/frontends/qt/InsetParamsDialog.cpp b/src/frontends/qt/InsetParamsDialog.cpp
index 3bccd2235ea7b35165f86f2f8dcd1a351d4815ab..67cbfdd94e1f5182c5ad791a3c3f56f935fc17dd 100644 (file)
--- a/src/frontends/qt/InsetParamsDialog.cpp
+++ b/src/frontends/qt/InsetParamsDialog.cpp
@@ -80,6 +80,8 @@ InsetParamsDialog::InsetParamsDialog(GuiView & lv, InsetParamsWidget * widget)
        widget->dialogTitle()), d(new Private)
 {
        setupUi(this);
+       // Remove margins since the widget is embedded in dialog which provides them
+       widget->layout()->setContentsMargins(0, 0, 0, 0);
        setInsetParamsWidget(widget);
        immediateApplyCB->setChecked(false);
        synchronizedCB->setChecked(true);
@@ -87,6 +89,8 @@ InsetParamsDialog::InsetParamsDialog(GuiView & lv, InsetParamsWidget * widget)
        setFocusProxy(widget);
        newPB = buttonBox->addButton(qt_("Ne&w Inset"),
                             QDialogButtonBox::ActionRole);
+       // fix height to minimum
+       setFixedHeight(sizeHint().height());
 }
 
 InsetParamsDialog::~InsetParamsDialog()
@@ -193,14 +197,16 @@ docstring InsetParamsDialog::checkWidgets(bool immediate)
                ? d->widget_->creationCode() : LFUN_INSET_MODIFY;
        bool const lfun_ok = lyx::getStatus(FuncRequest(code, argument)).enabled();
 
+       bool const changed_inset = ins && (ins != d->inset_ || d->changed_);
        buttonBox->button(QDialogButtonBox::Ok)->setEnabled(!immediate && widget_ok
-                                                           && !read_only && valid_argument);
-       bool const can_be_restored = !immediate && !read_only
-                       && ins && (ins != d->inset_ || d->changed_);
+                                                           && !read_only && valid_argument
+                                                           && (!ins || changed_inset));
+       bool const can_be_restored = !immediate && !read_only && changed_inset;
        buttonBox->button(QDialogButtonBox::Reset)->setEnabled(can_be_restored);
-       buttonBox->button(QDialogButtonBox::Apply)->setEnabled(ins && !immediate
+       buttonBox->button(QDialogButtonBox::Apply)->setEnabled(!immediate
                                                               && lfun_ok && widget_ok
-                                                              && !read_only && valid_argument);
+                                                              && !read_only && valid_argument
+                                                              && changed_inset);
        immediateApplyCB->setEnabled(ins && !read_only);
        // This seems to be the only way to access custom buttons
        QList<QAbstractButton*> buttons = buttonBox->buttons();
@@ -259,72 +265,6 @@ void InsetParamsDialog::updateView()
        buttonBox->button(QDialogButtonBox::Ok)->setDefault(true);
 }
 
-
-Dialog * createDialog(GuiView & lv, InsetCode code)
-{
-       InsetParamsWidget * widget;
-       switch (code) {
-       case ERT_CODE:
-               widget = new GuiERT;
-               break;
-       case FLOAT_CODE:
-               widget = new FloatPlacement(true);
-               break;
-       case BIBITEM_CODE:
-               widget = new GuiBibitem;
-               break;
-       case BRANCH_CODE:
-               widget = new GuiBranch;
-               break;
-       case BOX_CODE:
-               widget = new GuiBox;
-               break;
-       case HYPERLINK_CODE:
-               widget = new GuiHyperlink;
-               break;
-       case COUNTER_CODE:
-               widget = new GuiCounter(lv, nullptr);
-               break;
-       case INFO_CODE:
-               widget = new GuiInfo;
-               break;
-       case LABEL_CODE:
-               widget = new GuiLabel;
-               break;
-       case LINE_CODE:
-               widget = new GuiLine;
-               break;
-       case MATH_SPACE_CODE:
-               widget = new GuiHSpace(true);
-               break;
-       case NOMENCL_CODE:
-               widget = new GuiNomenclature;
-               break;
-       case NOMENCL_PRINT_CODE:
-               widget = new GuiPrintNomencl;
-               break;
-       case SPACE_CODE:
-               widget = new GuiHSpace(false);
-               break;
-       case TABULAR_CODE:
-               widget = new GuiTabular;
-               break;
-       case VSPACE_CODE:
-               widget = new GuiVSpace;
-               break;
-       default:
-               return nullptr;
-       }
-       InsetParamsDialog * dialog = new InsetParamsDialog(lv, widget);
-       return dialog;
-}
-
-
-Dialog * createDialog(GuiView & lv, string const & name)
-{
-       return createDialog(lv, insetCode(name));
-}
-
 } // namespace frontend
 } // namespace lyx