[ui] Better visual queue for non-visible layers in the tree

nirvn · nirvn · commit c21597a327be · 2018-12-11T17:35:59.000+07:00
diff --git a/src/core/layertree/qgslayertreemodel.cpp b/src/core/layertree/qgslayertreemodel.cpp
@@ -260,6 +260,14 @@ QVariant QgsLayerTreeModel::data( const QModelIndex &index, int role ) const
     QFont f( QgsLayerTree::isLayer( node ) ? mFontLayer : ( QgsLayerTree::isGroup( node ) ? mFontGroup : QFont() ) );
     if ( index == mCurrentIndex )
       f.setUnderline( true );
+    if ( QgsLayerTree::isLayer( node ) )
+    {
+      const QgsMapLayer *layer = QgsLayerTree::toLayer( node )->layer();
+      if ( ( !node->isVisible() && ( !layer || layer->isSpatial() ) ) || ( layer && !layer->isInScaleRange( mLegendMapViewScale ) ) )
+      {
+        f.setItalic( !f.italic() );
+      }
+    }
     return f;
   }
   else if ( role == Qt::ForegroundRole )
@@ -270,7 +278,7 @@ QVariant QgsLayerTreeModel::data( const QModelIndex &index, int role ) const
       const QgsMapLayer *layer = QgsLayerTree::toLayer( node )->layer();
       if ( ( !node->isVisible() && ( !layer || layer->isSpatial() ) ) || ( layer && !layer->isInScaleRange( mLegendMapViewScale ) ) )
       {
-        brush.setColor( Qt::lightGray );
+        brush.setColor( Qt::gray );
       }
     }
     return brush;

Original file line number	Diff line number	Diff line change
`@@ -260,6 +260,14 @@ QVariant QgsLayerTreeModel::data( const QModelIndex &index, int role ) const`
`260`	`260`	`QFont f( QgsLayerTree::isLayer( node ) ? mFontLayer : ( QgsLayerTree::isGroup( node ) ? mFontGroup : QFont() ) );`
`261`	`261`	`if ( index == mCurrentIndex )`
`262`	`262`	`f.setUnderline( true );`
	`263`	`+ if ( QgsLayerTree::isLayer( node ) )`
	`264`	`+ {`
	`265`	`+ const QgsMapLayer *layer = QgsLayerTree::toLayer( node )->layer();`
	`266`	`+ if ( ( !node->isVisible() && ( !layer \|\| layer->isSpatial() ) ) \|\| ( layer && !layer->isInScaleRange( mLegendMapViewScale ) ) )`
	`267`	`+ {`
	`268`	`+ f.setItalic( !f.italic() );`
	`269`	`+ }`
	`270`	`+ }`
`263`	`271`	`return f;`
`264`	`272`	`}`
`265`	`273`	`else if ( role == Qt::ForegroundRole )`
`@@ -270,7 +278,7 @@ QVariant QgsLayerTreeModel::data( const QModelIndex &index, int role ) const`
`270`	`278`	`const QgsMapLayer *layer = QgsLayerTree::toLayer( node )->layer();`
`271`	`279`	`if ( ( !node->isVisible() && ( !layer \|\| layer->isSpatial() ) ) \|\| ( layer && !layer->isInScaleRange( mLegendMapViewScale ) ) )`
`272`	`280`	`{`
`273`		`- brush.setColor( Qt::lightGray );`
	`281`	`+ brush.setColor( Qt::gray );`
`274`	`282`	`}`
`275`	`283`	`}`
`276`	`284`	`return brush;`