[Pkg-owncloud-commits] [owncloud-client] 339/484: ShareDialog: remove the search button (#3737)

[Sandro Knauß]

This is an automated email from the git hooks/post-receive script.

hefee-guest pushed a commit to branch master
in repository owncloud-client.

commit 6e4240511354a6cf6cc47155911c2eb669a8fc75
Author: Olivier Goffart <ogoffart at woboq.com>
Date:   Tue Nov 17 10:24:35 2015 +0100

    ShareDialog:  remove the search button (#3737)
---
 src/gui/sharedialog.cpp          |  1 +
 src/gui/shareusergroupwidget.cpp | 13 +++----------
 src/gui/shareusergroupwidget.h   |  2 +-
 src/gui/shareusergroupwidget.ui  | 29 ++++++-----------------------
 4 files changed, 11 insertions(+), 34 deletions(-)

diff --git a/src/gui/sharedialog.cpp b/src/gui/sharedialog.cpp
index 0177e8e..0d5f3a0 100644
--- a/src/gui/sharedialog.cpp
+++ b/src/gui/sharedialog.cpp
@@ -46,6 +46,7 @@ ShareDialog::ShareDialog(AccountPtr account, const QString &sharePath, const QSt
 
     QPushButton *closeButton = _ui->buttonBox->button(QDialogButtonBox::Close);
     connect(closeButton, SIGNAL(clicked()), this, SLOT(close()));
+    closeButton->setDefault(false); // Because people press enter in the dialog and we don't want to close for that
 
     // Set icon
     QFileInfo f_info(_localPath);
diff --git a/src/gui/shareusergroupwidget.cpp b/src/gui/shareusergroupwidget.cpp
index 92357b8..0613037 100644
--- a/src/gui/shareusergroupwidget.cpp
+++ b/src/gui/shareusergroupwidget.cpp
@@ -65,8 +65,6 @@ ShareUserGroupWidget::ShareUserGroupWidget(AccountPtr account, const QString &sh
     _completer->setModel(_completerModel);
     _ui->shareeLineEdit->setCompleter(_completer);
 
-    _ui->searchPushButton->setEnabled(false);
-
     _manager = new ShareManager(_account, this);
     connect(_manager, SIGNAL(sharesFetched(QList<QSharedPointer<Share>>)), SLOT(slotSharesFetched(QList<QSharedPointer<Share>>)));
     connect(_manager, SIGNAL(shareCreated(QSharedPointer<Share>)), SLOT(getShares()));
@@ -76,7 +74,7 @@ ShareUserGroupWidget::ShareUserGroupWidget(AccountPtr account, const QString &sh
     // Queued connection so this signal is recieved after textChanged
     connect(_ui->shareeLineEdit, SIGNAL(textEdited(QString)),
             this, SLOT(slotLineEditTextEdited(QString)), Qt::QueuedConnection);
-    connect(&_completionTimer, SIGNAL(timeout()), this, SLOT(on_searchPushButton_clicked()));
+    connect(&_completionTimer, SIGNAL(timeout()), this, SLOT(searchForSharees()));
     _completionTimer.setSingleShot(true);
     _completionTimer.setInterval(600);
 
@@ -88,14 +86,9 @@ ShareUserGroupWidget::~ShareUserGroupWidget()
     delete _ui;
 }
 
-void ShareUserGroupWidget::on_shareeLineEdit_textChanged(const QString &text)
+void ShareUserGroupWidget::on_shareeLineEdit_textChanged(const QString &)
 {
     _completionTimer.stop();
-    if (text == "") {
-        _ui->searchPushButton->setEnabled(false);
-    } else {
-        _ui->searchPushButton->setEnabled(true);
-    }
 }
 
 void ShareUserGroupWidget::slotLineEditTextEdited(const QString& text)
@@ -108,7 +101,7 @@ void ShareUserGroupWidget::slotLineEditTextEdited(const QString& text)
 }
 
 
-void ShareUserGroupWidget::on_searchPushButton_clicked()
+void ShareUserGroupWidget::searchForSharees()
 {
     _completionTimer.stop();
     ShareeModel::ShareeSet blacklist;
diff --git a/src/gui/shareusergroupwidget.h b/src/gui/shareusergroupwidget.h
index fa1bd41..13b3ef8 100644
--- a/src/gui/shareusergroupwidget.h
+++ b/src/gui/shareusergroupwidget.h
@@ -97,7 +97,7 @@ private slots:
     void slotSharesFetched(const QList<QSharedPointer<Share>> &shares);
 
     void on_shareeLineEdit_textChanged(const QString &text);
-    void on_searchPushButton_clicked();
+    void searchForSharees();
     void slotLineEditTextEdited(const QString &text);
 
     void slotCompleterActivated(const QModelIndex & index);
diff --git a/src/gui/shareusergroupwidget.ui b/src/gui/shareusergroupwidget.ui
index 5b8e0f0..e4c4040 100644
--- a/src/gui/shareusergroupwidget.ui
+++ b/src/gui/shareusergroupwidget.ui
@@ -14,29 +14,12 @@
    <string>Share NewDocument.odt</string>
   </property>
   <layout class="QVBoxLayout" name="verticalLayout">
-   <property name="sizeConstraint">
-    <enum>QLayout::SetDefaultConstraint</enum>
-   </property>
    <item>
-    <layout class="QHBoxLayout" name="horizontalLayout">
-     <item>
-      <widget class="QLineEdit" name="shareeLineEdit">
-       <property name="placeholderText">
-        <string>Share with users, groups or remote users ...</string>
-       </property>
-      </widget>
-     </item>
-     <item>
-      <widget class="QPushButton" name="searchPushButton">
-       <property name="text">
-        <string>Search</string>
-       </property>
-       <property name="default">
-        <bool>true</bool>
-       </property>
-      </widget>
-     </item>
-    </layout>
+    <widget class="QLineEdit" name="shareeLineEdit">
+     <property name="placeholderText">
+      <string>Share with users, groups or remote users ...</string>
+     </property>
+    </widget>
    </item>
    <item>
     <widget class="QScrollArea" name="scrollArea">
@@ -61,7 +44,7 @@
         <x>0</x>
         <y>0</y>
         <width>441</width>
-        <height>124</height>
+        <height>128</height>
        </rect>
       </property>
       <layout class="QVBoxLayout" name="verticalLayout_3"/>

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-owncloud/owncloud-client.git