Skip to content

Commit

Permalink
Disable preview modes under OSX due to issues with QGraphicsEffects u…
Browse files Browse the repository at this point in the history
…nder OSX (refs #10761)
  • Loading branch information
nyalldawson committed Jul 27, 2014
1 parent bd5846b commit 5ecc908
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 2 deletions.
4 changes: 4 additions & 0 deletions src/app/composer/qgscomposer.cpp
Expand Up @@ -296,12 +296,16 @@ QgsComposer::QgsComposer( QgisApp *qgis, const QString& title )
QShortcut* ctrlEquals = new QShortcut( QKeySequence( "Ctrl+=" ), this );
connect( ctrlEquals, SIGNAL( activated() ), mActionZoomIn, SLOT( trigger() ) );

#ifndef Q_WS_MAC
//disabled for OSX - see #10761
//also see http://qt-project.org/forums/viewthread/3630 QGraphicsEffects are not well supported on OSX
QMenu *previewMenu = viewMenu->addMenu( "Preview" );
previewMenu->addAction( mActionPreviewModeOff );
previewMenu->addAction( mActionPreviewModeGrayscale );
previewMenu->addAction( mActionPreviewModeMono );
previewMenu->addAction( mActionPreviewProtanope );
previewMenu->addAction( mActionPreviewDeuteranope );
#endif

viewMenu->addSeparator();
viewMenu->addAction( mActionZoomIn );
Expand Down
8 changes: 8 additions & 0 deletions src/app/qgisapp.cpp
Expand Up @@ -1471,6 +1471,14 @@ void QgisApp::createMenus()
mSettingsMenu->insertSeparator( before );
}


#ifdef Q_WS_MAC
//disabled for OSX - see #10761
//also see http://qt-project.org/forums/viewthread/3630 QGraphicsEffects are not well supported on OSX
mMenuPreviewMode->menuAction()->setVisible( false );
#endif


#ifdef Q_WS_MAC

// keep plugins from hijacking About and Preferences application menus
Expand Down
4 changes: 2 additions & 2 deletions src/ui/qgisapp.ui
Expand Up @@ -88,7 +88,7 @@
<addaction name="mActionDecorationNorthArrow"/>
<addaction name="mActionDecorationCopyright"/>
</widget>
<widget class="QMenu" name="menuPreview_Mode">
<widget class="QMenu" name="mMenuPreviewMode">
<property name="title">
<string>Preview Mode</string>
</property>
Expand All @@ -115,7 +115,7 @@
<addaction name="mActionZoomActualSize"/>
<addaction name="separator"/>
<addaction name="menuDecorations"/>
<addaction name="menuPreview_Mode"/>
<addaction name="mMenuPreviewMode"/>
<addaction name="mActionMapTips"/>
<addaction name="mActionNewBookmark"/>
<addaction name="mActionShowBookmarks"/>
Expand Down

0 comments on commit 5ecc908

Please sign in to comment.