Merge pull request #798 from 3nids/maptoolcapturerubber

[fix #8395] Add a second rubber band in map tool capture

Author: NathanW2

src/app/qgsmaptooladdfeature.cpp

@@ -141,6 +141,8 @@ void QgsMapToolAddFeature::canvasReleaseEvent( QMouseEvent * e )
       mCanvas->refresh();
     }
   }
+
+  // LINE AND POLYGON CAPTURING
   else if ( mode() == CaptureLine || mode() == CapturePolygon )
   {
     //check we only use the line tool for line/multiline layers
@@ -181,8 +183,7 @@ void QgsMapToolAddFeature::canvasReleaseEvent( QMouseEvent * e )
     else if ( e->button() == Qt::RightButton )
     {
       // End of string
-
-      resetLastVertex();
+      deleteTempRubberBand();
 
       //lines: bail out if there are not at least two vertices
       if ( mode() == CaptureLine && size() < 2 )

src/app/qgsmaptooladdpart.cpp

@@ -112,7 +112,7 @@ void QgsMapToolAddPart::canvasReleaseEvent( QMouseEvent * e )
       }
       else if ( e->button() != Qt::RightButton )
       {
-        resetLastVertex();
+        deleteTempRubberBand();
 
         return;
       }

src/app/qgsmaptooladdring.cpp

@@ -69,7 +69,7 @@ void QgsMapToolAddRing::canvasReleaseEvent( QMouseEvent * e )
   }
   else if ( e->button() == Qt::RightButton )
   {
-    resetLastVertex();
+    deleteTempRubberBand();
 
     closePolygon();
 

src/app/qgsmaptoolcapture.cpp

@@ -36,6 +36,7 @@ QgsMapToolCapture::QgsMapToolCapture( QgsMapCanvas* canvas, enum CaptureMode too
     : QgsMapToolEdit( canvas )
     , mCaptureMode( tool )
     , mRubberBand( 0 )
+    , mTempRubberBand( 0 )
     , mValidator( 0 )
 {
   mCaptureModeFromLayer = tool == CaptureNone;
@@ -100,7 +101,6 @@ void QgsMapToolCapture::currentLayerChanged( QgsMapLayer *layer )
   }
 }
 
-
 void QgsMapToolCapture::canvasMoveEvent( QMouseEvent * e )
 {
   QgsPoint mapPoint;
@@ -120,10 +120,10 @@ void QgsMapToolCapture::canvasMoveEvent( QMouseEvent * e )
       mSnappingMarkers << m;
     }
 
-    if ( mCaptureMode != CapturePoint && mRubberBand && mCapturing )
+    if ( mCaptureMode != CapturePoint && mTempRubberBand && mCapturing )
     {
       mapPoint = snapPointFromResults( snapResults, e->pos() );
-      mRubberBand->movePoint( mapPoint );
+      mTempRubberBand->movePoint( mapPoint );
     }
   }
 } // mouseMoveEvent
@@ -180,7 +180,6 @@ int QgsMapToolCapture::nextPoint( const QPoint &p, QgsPoint &layerPoint, QgsPoin
   return 0;
 }
 
-
 int QgsMapToolCapture::addVertex( const QPoint &p )
 {
   QgsPoint layerPoint;
@@ -203,10 +202,28 @@ int QgsMapToolCapture::addVertex( const QPoint &p )
   {
     mRubberBand = createRubberBand( mCaptureMode == CapturePolygon ? QGis::Polygon : QGis::Line );
   }
-
   mRubberBand->addPoint( mapPoint );
   mCaptureList.append( layerPoint );
 
+  if ( !mTempRubberBand )
+  {
+    mTempRubberBand = createRubberBand( mCaptureMode == CapturePolygon ? QGis::Polygon : QGis::Line , true );
+  }
+  else{
+    mTempRubberBand->reset(CapturePolygon ? true : false);
+  }
+  if ( mCaptureMode == CaptureLine )
+  {
+    mTempRubberBand->addPoint( mapPoint );
+  }
+  else if ( mCaptureMode == CapturePolygon )
+  {
+    const QgsPoint *firstPoint = mRubberBand->getPoint( 0 , 0 );
+    mTempRubberBand->addPoint( *firstPoint );
+    mTempRubberBand->movePoint( mapPoint );
+    mTempRubberBand->addPoint( mapPoint );
+  }
+
   validateGeometry();
 
   return 0;
@@ -231,20 +248,6 @@ void QgsMapToolCapture::undo()
   }
 }
 
-void QgsMapToolCapture::resetLastVertex()
-{
-  if ( mRubberBand )
-  {
-    int rubberBandSize = mRubberBand->numberOfVertices();
-    if ( rubberBandSize < 2 )
-    {
-      return;
-    }
-    const QgsPoint *lastPoint = mRubberBand->getPoint(0, rubberBandSize-2);
-    mRubberBand->movePoint( *lastPoint );
-  }
-}
-
 void QgsMapToolCapture::keyPressEvent( QKeyEvent* e )
 {
   if ( e->key() == Qt::Key_Backspace || e->key() == Qt::Key_Delete )
@@ -266,6 +269,12 @@ void QgsMapToolCapture::stopCapturing()
     mRubberBand = 0;
   }
 
+  if ( mTempRubberBand )
+  {
+    delete mTempRubberBand;
+    mTempRubberBand = 0;
+  }
+
   while ( !mGeomErrorMarkers.isEmpty() )
   {
     delete mGeomErrorMarkers.takeFirst();
@@ -283,6 +292,15 @@ void QgsMapToolCapture::stopCapturing()
   mCanvas->refresh();
 }
 
+void QgsMapToolCapture::deleteTempRubberBand()
+{
+  if ( mTempRubberBand )
+  {
+    delete mTempRubberBand;
+    mTempRubberBand = 0;
+  }
+}
+
 void QgsMapToolCapture::closePolygon()
 {
   mCaptureList.append( mCaptureList[0] );

src/app/qgsmaptoolcapture.h

@@ -82,11 +82,9 @@ class QgsMapToolCapture : public QgsMapToolEdit
     /**Removes the last vertex from mRubberBand and mCaptureList*/
     void undo();
 
-    /**Reset the last vertex from RubberBand to the previous one position*/
-    void resetLastVertex();
-
     void startCapturing();
     void stopCapturing();
+    void deleteTempRubberBand();
 
     CaptureMode mode() { return mCaptureMode; }
 
@@ -107,6 +105,9 @@ class QgsMapToolCapture : public QgsMapToolEdit
     /** rubber band for polylines and polygons */
     QgsRubberBand* mRubberBand;
 
+    /** temporary rubber band for polylines and polygons. this connects the last added point to the mouse cursor position */
+    QgsRubberBand* mTempRubberBand;
+
     /** List to store the points of digitised lines and polygons (in layer coordinates)*/
     QList<QgsPoint> mCaptureList;
 

src/app/qgsmaptooledit.cpp

@@ -68,15 +68,26 @@ QgsPoint QgsMapToolEdit::snapPointFromResults( const QList<QgsSnappingResult>& s
   }
 }
 
-QgsRubberBand* QgsMapToolEdit::createRubberBand( QGis::GeometryType geometryType )
+QgsRubberBand* QgsMapToolEdit::createRubberBand( QGis::GeometryType geometryType, bool alternativeBand )
 {
   QSettings settings;
   QgsRubberBand* rb = new QgsRubberBand( mCanvas, geometryType );
+  rb->setWidth( settings.value( "/qgis/digitizing/line_width", 1 ).toInt() );
   QColor color( settings.value( "/qgis/digitizing/line_color_red", 255 ).toInt(),
                 settings.value( "/qgis/digitizing/line_color_green", 0 ).toInt(),
-                settings.value( "/qgis/digitizing/line_color_blue", 0 ).toInt() );
+                settings.value( "/qgis/digitizing/line_color_blue", 0 ).toInt());
+  double myAlpha = settings.value( "/qgis/digitizing/line_color_alpha", 200 ).toInt() / 255.0 ;
+  if ( alternativeBand )
+  {
+    myAlpha = myAlpha * settings.value( "/qgis/digitizing/line_color_alpha_scale" , 0.75 ).toDouble();
+    rb->setLineStyle( Qt::DotLine );
+  }
+  if ( geometryType == QGis::Polygon )
+  {
+    color.setAlphaF ( myAlpha );
+  }
+  color.setAlphaF ( myAlpha );
   rb->setColor( color );
-  rb->setWidth( settings.value( "/qgis/digitizing/line_width", 1 ).toInt() );
   rb->show();
   return rb;
 }

src/app/qgsmaptooledit.h

@@ -50,10 +50,13 @@ class QgsMapToolEdit: public QgsMapTool
        @return the snapped point in map coordinates*/
     QgsPoint snapPointFromResults( const QList<QgsSnappingResult>& snapResults, const QPoint& screenCoords );
 
-    /**Creates a rubber band with the color/line width from
-     the QGIS settings. The caller takes ownership of the
-    returned object*/
-    QgsRubberBand* createRubberBand( QGis::GeometryType geometryType = QGis::Line );
+    /** Creates a rubber band with the color/line width from
+    *   the QGIS settings. The caller takes ownership of the
+    *   returned object
+    *   @param geometryType
+    *   @param alternativeBand if true, rubber band will be set with more transparency and a dash pattern. defaut is false.
+    */
+    QgsRubberBand* createRubberBand(QGis::GeometryType geometryType = QGis::Line , bool alternativeBand = false );
 
     /**Returns the current vector layer of the map canvas or 0*/
     QgsVectorLayer* currentVectorLayer();

src/app/qgsmaptoolreshape.cpp

@@ -68,7 +68,7 @@ void QgsMapToolReshape::canvasReleaseEvent( QMouseEvent * e )
   }
   else if ( e->button() == Qt::RightButton )
   {
-    resetLastVertex();
+    deleteTempRubberBand();
 
     //find out bounding box of mCaptureList
     if ( size() < 1 )

src/app/qgsmaptoolsplitfeatures.cpp

@@ -74,7 +74,7 @@ void QgsMapToolSplitFeatures::canvasReleaseEvent( QMouseEvent * e )
   }
   else if ( e->button() == Qt::RightButton )
   {
-    resetLastVertex();
+    deleteTempRubberBand();
 
     //bring up dialog if a split was not possible (polygon) or only done once (line)
     int topologicalEditing = QgsProject::instance()->readNumEntry( "Digitizing", "/TopologicalEditing", 0 );

src/app/qgsoptions.cpp

@@ -638,7 +638,9 @@ QgsOptions::QgsOptions( QWidget *parent, Qt::WFlags fl ) :
   myRed = settings.value( "/qgis/digitizing/line_color_red", 255 ).toInt();
   myGreen = settings.value( "/qgis/digitizing/line_color_green", 0 ).toInt();
   myBlue = settings.value( "/qgis/digitizing/line_color_blue", 0 ).toInt();
-  mLineColorToolButton->setColor( QColor( myRed, myGreen, myBlue ) );
+  myAlpha = settings.value( "/qgis/digitizing/line_color_alpha", 200 ).toInt();
+  mLineColorToolButton->setColor( QColor( myRed, myGreen, myBlue, myAlpha ) );
+  mLineColorToolButton->setColorDialogOptions(QColorDialog::ShowAlphaChannel);
 
   //default snap mode
   mDefaultSnapModeComboBox->insertItem( 0, tr( "To vertex" ), "to vertex" );
@@ -1083,6 +1085,7 @@ void QgsOptions::saveOptions()
   settings.setValue( "/qgis/digitizing/line_color_red", digitizingColor.red() );
   settings.setValue( "/qgis/digitizing/line_color_green", digitizingColor.green() );
   settings.setValue( "/qgis/digitizing/line_color_blue", digitizingColor.blue() );
+  settings.setValue( "/qgis/digitizing/line_color_alpha", digitizingColor.alpha() );
 
   //default snap mode
   QString defaultSnapModeString = mDefaultSnapModeComboBox->itemData( mDefaultSnapModeComboBox->currentIndex() ).toString();