Skip to content

Commit

Permalink
Added warning to fullscreen mode for Linux as this doesn't currently …
Browse files Browse the repository at this point in the history
…work. Left in a number of commented approached I already tried. This is worth revisiting when Linux uses a modern post-ICCCM specification windows manager
  • Loading branch information
RussellGarwood committed Oct 21, 2018
1 parent 3626c06 commit bc451d6
Showing 1 changed file with 12 additions and 2 deletions.
14 changes: 12 additions & 2 deletions src/mainwindow.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3876,12 +3876,21 @@ void MainWindow::updateScreenRatio()
*/
void MainWindow::on_actionFull_Screen_triggered()
{

#ifdef __linux__
QMessageBox::warning(this, "Apologies", "Full screen doesn't currently work on Linux due to limitations of the QWidget::showFullScreen() call. Sorry about that.");
return;
//fullScreenDialog->setWindowFlags( Qt::X11BypassWindowManagerHint);
// fullScreenDialog->setWindowFlags(Qt::X11BypassWindowManagerHint | Qt::WindowStaysOnTopHint | Qt::FramelessWindowHint);
//fullScreenDialog->showMaximized();
//fullScreenDialog->setWindowState(Qt::WindowFullScreen);
//fullScreenDialog->show();
#else
if (isGLFullScreen == false)
{
fullScreenDialog = new FullScreenWindow(this, gl3widget);
fullScreenDialog->showFullScreen();
//fullScreenDialog->show();

fullScreenDialog->showFullScreen();
isGLFullScreen = true;

//qDebug() << "[Full Screen Mode] Opening full screen mode";
Expand All @@ -3896,4 +3905,5 @@ void MainWindow::on_actionFull_Screen_triggered()

//qDebug() << "[Full Screen Mode] Closing full screen mode";
}
#endif
}

0 comments on commit bc451d6

Please sign in to comment.