Apply patch from gcarillo to fix bug #1793

mhugent · mhugent · commit 556be42c426f · 2009-08-03T06:50:58.000Z
git-svn-id: http://svn.osgeo.org/qgis/trunk@11255 c8812cc2-4d05-0410-92ff-de0c093fc19c
diff --git a/src/app/qgscontinuouscolordialog.cpp b/src/app/qgscontinuouscolordialog.cpp
@@ -27,7 +27,7 @@
 #include "qgslogger.h"
 
 #include <QColorDialog>
-
+#include <QKeyEvent>
 
 QgsContinuousColorDialog::QgsContinuousColorDialog( QgsVectorLayer * layer )
     : QDialog(), mVectorLayer( layer )
@@ -203,3 +203,12 @@ void QgsContinuousColorDialog::on_cb_polygonOutline_clicked()
   else
     outlinewidthspinbox->setEnabled( false );
 }
+
+void QgsContinuousColorDialog::keyPressEvent( QKeyEvent * e )
+{
+  // Ignore the ESC key to avoid close the dialog without the properties window
+  if ( e->key() == Qt::Key_Escape )
+  {
+    e->ignore();
+  }
+}
diff --git a/src/app/qgscontinuouscolordialog.h b/src/app/qgscontinuouscolordialog.h
@@ -45,6 +45,9 @@ class QgsContinuousColorDialog: public QDialog, private Ui::QgsContinuousColorDi
   protected:
     QgsVectorLayer* mVectorLayer;
 
+    // Reimplements dialog keyPress event so we can ignore it
+    void keyPressEvent( QKeyEvent * event );
+
   private:
     /** Default constructor is private, do not use this */
     QgsContinuousColorDialog();
diff --git a/src/app/qgsgraduatedsymboldialog.cpp b/src/app/qgsgraduatedsymboldialog.cpp
@@ -28,6 +28,8 @@
 #include "qgsvectorlayer.h"
 #include "qgslogger.h"
 
+#include <QKeyEvent>
+
 QgsGraduatedSymbolDialog::QgsGraduatedSymbolDialog( QgsVectorLayer * layer ): QDialog(), mVectorLayer( layer ), sydialog( layer )
 {
   setupUi( this );
@@ -598,3 +600,12 @@ void QgsGraduatedSymbolDialog::updateEntryIcon( QgsSymbol * thepSymbol,
       ;
   }
 }
+
+void QgsGraduatedSymbolDialog::keyPressEvent( QKeyEvent * e )
+{
+  // Ignore the ESC key to avoid close the dialog without the properties window
+  if ( e->key() == Qt::Key_Escape )
+  {
+    e->ignore();
+  }
+}
diff --git a/src/app/qgsgraduatedsymboldialog.h b/src/app/qgsgraduatedsymboldialog.h
@@ -70,6 +70,9 @@ class QgsGraduatedSymbolDialog: public QDialog, private Ui::QgsGraduatedSymbolDi
     /**Gets the color value along a specified ramp**/
     QColor getColorFromRamp( QString ramp, int step, int totalSteps );
 
+    // Reimplements dialog keyPress event so we can ignore it
+    void keyPressEvent( QKeyEvent * event );
+
   protected slots:
     /**Removes a class from the classification*/
     void deleteCurrentClass();
diff --git a/src/app/qgssinglesymboldialog.cpp b/src/app/qgssinglesymboldialog.cpp
@@ -31,6 +31,7 @@
 #include <QImage>
 #include <QFileDialog>
 #include <QListWidgetItem>
+#include <QKeyEvent>
 
 #define DO_NOT_USE_STR "<off>"
 
@@ -636,3 +637,11 @@ void QgsSingleSymbolDialog::fillStyleChanged( int theIndex )
 
 }
 
+void QgsSingleSymbolDialog::keyPressEvent( QKeyEvent * e )
+{
+  // Ignore the ESC key to avoid close the dialog without the properties window
+  if ( e->key() == Qt::Key_Escape )
+  {
+    e->ignore();
+  }
+}
diff --git a/src/app/qgssinglesymboldialog.h b/src/app/qgssinglesymboldialog.h
@@ -50,6 +50,9 @@ class QgsSingleSymbolDialog: public QDialog, private Ui::QgsSingleSymbolDialogBa
     QgsVectorLayer* mVectorLayer;
     bool mDisabled;
 
+    // Reimplements dialog keyPress event so we can ignore it
+    void keyPressEvent( QKeyEvent * event );
+
   public slots:
     /* arrange the widgets on this dialog to reflect the current state of QgsSymbol */
     void unset();
diff --git a/src/app/qgsuniquevaluedialog.cpp b/src/app/qgsuniquevaluedialog.cpp
@@ -27,6 +27,7 @@
 #include "qgslogger.h"
 
 #include <QMessageBox>
+#include <QKeyEvent>
 
 QgsUniqueValueDialog::QgsUniqueValueDialog( QgsVectorLayer* vl ): QDialog(), mVectorLayer( vl ), sydialog( vl, true )
 {
@@ -458,3 +459,12 @@ void QgsUniqueValueDialog::updateEntryIcon( QgsSymbol * thepSymbol,
       //do nothing
   }
 }
+
+void QgsUniqueValueDialog::keyPressEvent( QKeyEvent * e )
+{
+  // Ignore the ESC key to avoid close the dialog without the properties window
+  if ( e->key() == Qt::Key_Escape )
+  {
+    e->ignore();
+  }
+}
diff --git a/src/app/qgsuniquevaluedialog.h b/src/app/qgsuniquevaluedialog.h
@@ -46,6 +46,9 @@ class QgsUniqueValueDialog: public QDialog, private Ui::QgsUniqueValueDialogBase
     QMap<QString, QgsSymbol*> mValues;
     QgsSingleSymbolDialog sydialog;
 
+    // Reimplements dialog keyPress event so we can ignore it
+    void keyPressEvent( QKeyEvent * event );
+
   protected slots:
     /**Set new attribut for classification*/
     void changeClassificationAttribute();

Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,8 @@`
`28`	`28`	`#include "qgsvectorlayer.h"`
`29`	`29`	`#include "qgslogger.h"`
`30`	`30`
	`31`	`+#include <QKeyEvent>`
	`32`	`+`
`31`	`33`	`QgsGraduatedSymbolDialog::QgsGraduatedSymbolDialog( QgsVectorLayer * layer ): QDialog(), mVectorLayer( layer ), sydialog( layer )`
`32`	`34`	`{`
`33`	`35`	`setupUi( this );`
`@@ -598,3 +600,12 @@ void QgsGraduatedSymbolDialog::updateEntryIcon( QgsSymbol * thepSymbol,`
`598`	`600`	`;`
`599`	`601`	`}`
`600`	`602`	`}`
	`603`	`+`
	`604`	`+void QgsGraduatedSymbolDialog::keyPressEvent( QKeyEvent * e )`
	`605`	`+{`
	`606`	`+ // Ignore the ESC key to avoid close the dialog without the properties window`
	`607`	`+ if ( e->key() == Qt::Key_Escape )`
	`608`	`+ {`
	`609`	`+ e->ignore();`
	`610`	`+ }`
	`611`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -27,6 +27,7 @@`
`27`	`27`	`#include "qgslogger.h"`
`28`	`28`
`29`	`29`	`#include <QMessageBox>`
	`30`	`+#include <QKeyEvent>`
`30`	`31`
`31`	`32`	`QgsUniqueValueDialog::QgsUniqueValueDialog( QgsVectorLayer* vl ): QDialog(), mVectorLayer( vl ), sydialog( vl, true )`
`32`	`33`	`{`
`@@ -458,3 +459,12 @@ void QgsUniqueValueDialog::updateEntryIcon( QgsSymbol * thepSymbol,`
`458`	`459`	`//do nothing`
`459`	`460`	`}`
`460`	`461`	`}`
	`462`	`+`
	`463`	`+void QgsUniqueValueDialog::keyPressEvent( QKeyEvent * e )`
	`464`	`+{`
	`465`	`+ // Ignore the ESC key to avoid close the dialog without the properties window`
	`466`	`+ if ( e->key() == Qt::Key_Escape )`
	`467`	`+ {`
	`468`	`+ e->ignore();`
	`469`	`+ }`
	`470`	`+}`