Author: vfr
Date: Sat Mar 3 15:39:32 2012
New Revision: 40839
URL: http://www.lyx.org/trac/changeset/40839
Log:
Fix handling of the add branch textfield in GuiBranches
- Enter in the textfield adds the branch,
- Make sure the dialog is not closed when pressing enter,
- Pressing Ctrl+Enter or the Enter on the numpad closes the dialog.
Patch based on a patch from Scott Kostyshak.
Modified:
lyx-devel/trunk/src/frontends/qt4/GuiBranches.cpp
lyx-devel/trunk/src/frontends/qt4/GuiBranches.h
lyx-devel/trunk/src/frontends/qt4/GuiDocument.cpp
lyx-devel/trunk/src/frontends/qt4/ui/BranchesUi.ui
Modified: lyx-devel/trunk/src/frontends/qt4/GuiBranches.cpp
==============================================================================
--- lyx-devel/trunk/src/frontends/qt4/GuiBranches.cpp Sat Mar 3 14:13:58
2012 (r40838)
+++ lyx-devel/trunk/src/frontends/qt4/GuiBranches.cpp Sat Mar 3 15:39:32
2012 (r40839)
@@ -28,6 +28,7 @@
#include "support/gettext.h"
#include "support/lstrings.h"
+#include <QKeyEvent>
#include <QListWidget>
#include <QTreeWidget>
#include <QTreeWidgetItem>
@@ -71,9 +72,39 @@
connect(undef_->cancelPB, SIGNAL(clicked()),
undef_, SLOT(reject()));
+ newBranchLE->installEventFilter(this);
newBranchLE->setValidator(new NoNewLineValidator(newBranchLE));
}
+
+bool GuiBranches::eventFilter(QObject * obj, QEvent * event)
+{
+ QEvent::Type etype = event->type();
+ if (etype == QEvent::KeyPress
+ && obj == newBranchLE
+ && addBranchPB->isEnabled()) {
+ QKeyEvent * keyEvent = static_cast<QKeyEvent *>(event);
+ int const keyPressed = keyEvent->key();
+ Qt::KeyboardModifiers const keyModifiers =
keyEvent->modifiers();
+
+ if (keyPressed == Qt::Key_Enter || keyPressed ==
Qt::Key_Return) {
+ if (!keyModifiers) {
+ on_addBranchPB_pressed();
+ } else if (keyModifiers == Qt::ControlModifier
+ || keyModifiers == Qt::KeypadModifier
+ || keyModifiers == (Qt::ControlModifier |
Qt::KeypadModifier)) {
+ on_addBranchPB_pressed();
+ newBranchLE->clearFocus();
+ okPressed();
+ }
+ event->accept();
+ return true;
+ }
+ }
+ return QObject::eventFilter(obj, event);
+}
+
+
void GuiBranches::update(BufferParams const & params)
{
branchlist_ = params.branchlist();
@@ -132,14 +163,20 @@
}
+void GuiBranches::on_newBranchLE_textChanged(QString)
+{
+ QString const new_branch = newBranchLE->text();
+ addBranchPB->setEnabled(!new_branch.isEmpty());
+}
+
+
void GuiBranches::on_addBranchPB_pressed()
{
QString const new_branch = newBranchLE->text();
- if (!new_branch.isEmpty()) {
- branchlist_.add(qstring_to_ucs4(new_branch));
- newBranchLE->clear();
- updateView();
- }
+ branchlist_.add(qstring_to_ucs4(new_branch));
+ newBranchLE->clear();
+ addBranchPB->setEnabled(false);
+ updateView();
}
Modified: lyx-devel/trunk/src/frontends/qt4/GuiBranches.h
==============================================================================
--- lyx-devel/trunk/src/frontends/qt4/GuiBranches.h Sat Mar 3 14:13:58
2012 (r40838)
+++ lyx-devel/trunk/src/frontends/qt4/GuiBranches.h Sat Mar 3 15:39:32
2012 (r40839)
@@ -49,9 +49,12 @@
void apply(BufferParams & params) const;
void setUnknownBranches(QStringList const & b) { unknown_branches_ = b;
}
+ bool eventFilter(QObject * obj, QEvent * event);
+
Q_SIGNALS:
void changed();
void renameBranches(docstring const &, docstring const &);
+ void okPressed();
protected:
void toggleBranch(QTreeWidgetItem *);
@@ -60,6 +63,7 @@
void updateView();
protected Q_SLOTS:
+ void on_newBranchLE_textChanged(QString);
void on_addBranchPB_pressed();
void on_removePB_pressed();
void on_renamePB_pressed();
Modified: lyx-devel/trunk/src/frontends/qt4/GuiDocument.cpp
==============================================================================
--- lyx-devel/trunk/src/frontends/qt4/GuiDocument.cpp Sat Mar 3 14:13:58
2012 (r40838)
+++ lyx-devel/trunk/src/frontends/qt4/GuiDocument.cpp Sat Mar 3 15:39:32
2012 (r40839)
@@ -1289,6 +1289,7 @@
this, SLOT(change_adaptor()));
connect(branchesModule, SIGNAL(renameBranches(docstring const &,
docstring const &)),
this, SLOT(branchesRename(docstring const &, docstring const
&)));
+ connect(branchesModule, SIGNAL(okPressed()), this, SLOT(slotOK()));
updateUnknownBranches();
Modified: lyx-devel/trunk/src/frontends/qt4/ui/BranchesUi.ui
==============================================================================
--- lyx-devel/trunk/src/frontends/qt4/ui/BranchesUi.ui Sat Mar 3 14:13:58
2012 (r40838)
+++ lyx-devel/trunk/src/frontends/qt4/ui/BranchesUi.ui Sat Mar 3 15:39:32
2012 (r40839)
@@ -103,6 +103,9 @@
</item>
<item row="0" column="3" >
<widget class="QPushButton" name="addBranchPB" >
+ <property name="enabled">
+ <bool>false</bool>
+ </property>
<property name="toolTip" >
<string>Add a new branch to the list</string>
</property>
