line edit: do not display clear icon when read only

qgis · Sep 24, 2014 · ce486da · ce486da
1 parent e194423
commit ce486da
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 6 deletions.
diff --git a/src/gui/editorwidgets/qgstexteditwrapper.cpp b/src/gui/editorwidgets/qgstexteditwrapper.cpp
@@ -139,5 +139,11 @@ void QgsTextEditWrapper::setEnabled( bool enabled )
     mPlainTextEdit->setReadOnly( !enabled );
 
   if ( mLineEdit )
-    mLineEdit->setReadOnly( !enabled );
+  {
+    QgsFilterLineEdit* qgsWidget = dynamic_cast<QgsFilterLineEdit*>( mLineEdit );
+    if ( qgsWidget )
+      qgsWidget->setReadOnly( !enabled );
+    else
+      mLineEdit->setReadOnly( !enabled );
+  }
 }
diff --git a/src/gui/qgsfilterlineedit.cpp b/src/gui/qgsfilterlineedit.cpp
@@ -81,15 +81,19 @@ void QgsFilterLineEdit::clear()
 void QgsFilterLineEdit::changeEvent( QEvent *e )
 {
   QLineEdit::changeEvent( e );
-  if ( !isEnabled() || isReadOnly() )
-    btnClear->setVisible( false );
-  else
-    btnClear->setVisible( text() != mNullValue );
+  btnClear->setVisible( isEnabled() && !isReadOnly() && !isNull() );
 }
 
+void QgsFilterLineEdit::setReadOnly( bool readOnly )
+{
+  QLineEdit::setReadOnly( readOnly );
+  btnClear->setVisible( isEnabled() && !isReadOnly() && !isNull() );
+}
+
+
 void QgsFilterLineEdit::onTextChanged( const QString &text )
 {
-  btnClear->setVisible( !isEnabled() && !isReadOnly() && text != mNullValue );
+  btnClear->setVisible( isEnabled() && !isReadOnly() && !isNull() );
 
   if ( isNull() )
   {

diff --git a/src/gui/qgsfilterlineedit.h b/src/gui/qgsfilterlineedit.h
@@ -58,6 +58,12 @@ class GUI_EXPORT QgsFilterLineEdit : public QLineEdit
      */
     inline bool isNull() const { return text() == mNullValue; }
 
+    /**
+     * @brief setReadOnly set the line edit to be read only and hide the clear button
+     * @note Since QLineEdit::setReadOnly() is not virtual, it needs to be called for QgsFilterLineEdit
+     */
+    void setReadOnly( bool readOnly );
+
   signals:
     void cleared();