← Back to team overview

linux-traipu team mailing list archive

[Bug 348934] Re: MythTV doesn't go full screen without desktop visual effects off

 

It was fixed upstream, but then reverted.

$ git show 3ad6c81aa83491b7e6d2d54c15daf16894a72299
commit 3ad6c81aa83491b7e6d2d54c15daf16894a72299
Author: Mark Kendall <mkendall@xxxxxxxxxx>
Date:   Wed Mar 16 11:53:06 2011 +0800

    libmythui: Fix fullscreen windowing with Compiz (and Gnome?)
    
    This is based on the patch from Lawrence Rust in ticket #9589 which
    notes that Compiz requires the window to be visible before we set its
    state.
    
    Testing across various Ubuntu desktops (with and without Compiz and
    composite enabled) and on OSX and windows suggests that it is safe just
    to show the window earlier in the initialisation process and avoid the
    ifdef.

diff --git a/mythtv/libs/libmythui/mythmainwindow.cpp b/mythtv/libs/libmythui/mythmainwindow.cpp
index 1d4e046..6e917cb 100644
--- a/mythtv/libs/libmythui/mythmainwindow.cpp
+++ b/mythtv/libs/libmythui/mythmainwindow.cpp
@@ -915,6 +915,7 @@ void MythMainWindow::Init(void)
 #endif
 
     setWindowFlags(flags);
+    Show();
 
     if (d->does_fill_screen && !GetMythUI()->IsGeometryOverridden())
     {
@@ -929,7 +930,6 @@ void MythMainWindow::Init(void)
     setFixedSize(QSize(d->screenwidth, d->screenheight));
 
     GetMythUI()->ThemeWidget(this);
-    Show();
 
     if (!GetMythDB()->GetNumSetting("HideMouseCursor", 0))
         setMouseTracking(true); // Required for mouse cursor auto-hide


$ git show 003fc8fb14acc9a6c4aeea2001d33d4cdb3c253d
commit 003fc8fb14acc9a6c4aeea2001d33d4cdb3c253d
Author: Mark Kendall <mkendall@xxxxxxxxxx>
Date:   Sun Mar 20 11:21:26 2011 +0800

    libmythui: Revert recent Compiz/fullscreen window fixes.
    
    Some multi-screen setups (TwinView) are still broken despite an
    attempted fix. I can't honestly see any combination of initialisation
    sequence that fixes both the Compiz issue and doesn't break Twinview
    setups, so reverting until someone can figure out a better way.
    
    Refs #9589

diff --git a/mythtv/libs/libmythui/mythmainwindow.cpp b/mythtv/libs/libmythui/mythmainwindow.cpp
index f75e75d..1d4e046 100644
--- a/mythtv/libs/libmythui/mythmainwindow.cpp
+++ b/mythtv/libs/libmythui/mythmainwindow.cpp
@@ -916,21 +916,20 @@ void MythMainWindow::Init(void)
 
     setWindowFlags(flags);
 
-    d->screenRect = QRect(d->xbase, d->ybase, d->screenwidth, d->screenheight);
-    d->uiScreenRect = QRect(0, 0, d->screenwidth, d->screenheight);
-
-    Show();
-
     if (d->does_fill_screen && !GetMythUI()->IsGeometryOverridden())
     {
         VERBOSE(VB_GENERAL, "Using Full Screen Window");
         setWindowState(Qt::WindowFullScreen);
     }
 
+    d->screenRect = QRect(d->xbase, d->ybase, d->screenwidth, d->screenheight);
+    d->uiScreenRect = QRect(0, 0, d->screenwidth, d->screenheight);
+
     setGeometry(d->xbase, d->ybase, d->screenwidth, d->screenheight);
     setFixedSize(QSize(d->screenwidth, d->screenheight));
 
     GetMythUI()->ThemeWidget(this);
+    Show();
 
     if (!GetMythDB()->GetNumSetting("HideMouseCursor", 0))
         setMouseTracking(true); // Required for mouse cursor auto-hide

-- 
You received this bug notification because you are a member of UBUNTU -
AL - BR, which is subscribed to MythTV.
https://bugs.launchpad.net/bugs/348934

Title:
  MythTV doesn't go full screen without desktop visual effects off

Status in MythTV:
  Fix Released
Status in “mythtv” package in Ubuntu:
  Confirmed

Bug description:
  Binary package hint: mythtv

  If "Visual Effects" are enabled on the Gnome desktop (System -->
  Preferences --> Appearance --> Visual Effects), mythfrontend doesn't
  go full screen, but the Gnome panels show.

  $ lsb_release -r
  Release:	9.04

  $ uname -a
  Linux kuken 2.6.28-11-generic #37-Ubuntu SMP Mon Mar 23 16:40:00 UTC 2009 x86_64 GNU/Linux

  ~$ apt-cache policy mythtv-frontend 
  mythtv-frontend:
    Installed: 0.21.0+fixes19961-0ubuntu8

  Not sure if this is the fault of MythTV or Compiz though.

  
  Klem fra Nils

To manage notifications about this bug go to:
https://bugs.launchpad.net/mythtv/+bug/348934/+subscriptions