Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[3d view] harmonize panel icon size
  • Loading branch information
nirvn committed Oct 13, 2017
1 parent 9a3b547 commit 873b4b0
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions src/app/3d/qgs3dmapcanvasdockwidget.cpp
Expand Up @@ -35,6 +35,7 @@ Qgs3DMapCanvasDockWidget::Qgs3DMapCanvasDockWidget( QWidget *parent )
QWidget *contentsWidget = new QWidget( this );

QToolBar *toolBar = new QToolBar( contentsWidget );
toolBar->setIconSize( QSize( 16, 16 ) );

This comment has been minimized.

Copy link
@nyalldawson

nyalldawson Oct 13, 2017

Collaborator

Is this correct? I think it should be setIconSize( QgisApp::instance()->iconSize( true ) );

This comment has been minimized.

Copy link
@nirvn

nirvn Oct 13, 2017

Author Contributor

@nyalldawson , I applied the same logic as the layer tree panel: https://github.com/qgis/QGIS/blob/master/src/app/qgisapp.cpp#L3474

That said, I think your proposal is better. OK to change both this as well as the layer tree panel here?

This comment has been minimized.

Copy link
@nyalldawson

nyalldawson Oct 13, 2017

Collaborator

Yes please!

toolBar->addAction( QgsApplication::getThemeIcon( QStringLiteral( "mActionZoomFullExtent.svg" ) ), "Reset view", this, &Qgs3DMapCanvasDockWidget::resetView );
toolBar->addAction( QgsApplication::getThemeIcon( QStringLiteral( "mIconProperties.svg" ) ), "Configure", this, &Qgs3DMapCanvasDockWidget::configure );

Expand Down

0 comments on commit 873b4b0

Please sign in to comment.