]> git.sesse.net Git - vlc/blobdiff - modules/gui/qt4/main_interface.cpp
Cosmetic: don't access the same configuration item twice.
[vlc] / modules / gui / qt4 / main_interface.cpp
index 77982c11f8a98d1f343db33ec5371420b380d379..a7dcbf0c6c9be8822da7b748a185866abfe3e7e3 100644 (file)
@@ -6,6 +6,7 @@
  *
  * Authors: ClĂ©ment Stenac <zorglub@videolan.org>
  *          Jean-Baptiste Kempf <jb@videolan.org>
+ *          Ilkka Ollakka <ileoo@videolan.org>
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
  *****************************************************************************/
 
+#ifdef HAVE_CONFIG_H
+# include "config.h"
+#endif
+
 #include "qt4.hpp"
 #include "main_interface.hpp"
 #include "input_manager.hpp"
@@ -29,6 +34,8 @@
 #include "util/customwidgets.hpp"
 #include "dialogs_provider.hpp"
 #include "components/interface_widgets.hpp"
+#include "components/playlist/playlist.hpp"
+#include "dialogs/extended.hpp"
 #include "dialogs/playlist.hpp"
 #include "menus.hpp"
 
 #include <QLabel>
 #include <QSlider>
 #include <QWidgetAction>
+#if 0
+#include <QDockWidget>
+#endif
+#include <QToolBar>
+#include <QGroupBox>
+#include <QDate>
+#include <QProgressBar>
 
 #include <assert.h>
 #include <vlc_keys.h>
 #include <vlc_vout.h>
 
-#ifdef WIN32
-    #define PREF_W 410
-    #define PREF_H 151
-#else
-    #define PREF_W 400
-    #define PREF_H 140
-#endif
-
 #define SET_WIDTH(i,j) i->widgetSize.setWidth(j)
 #define SET_HEIGHT(i,j) i->widgetSize.setHeight(j)
 #define SET_WH( i,j,k) i->widgetSize.setWidth(j); i->widgetSize.setHeight(k);
@@ -70,126 +76,94 @@ static int IntfShowCB( vlc_object_t *p_this, const char *psz_variable,
                        vlc_value_t old_val, vlc_value_t new_val, void *param );
 static int InteractCallback( vlc_object_t *, const char *, vlc_value_t,
                              vlc_value_t, void *);
-/* Video handling */
-static void *DoRequest( intf_thread_t *p_intf, vout_thread_t *p_vout,
-                        int *pi1, int *pi2, unsigned int*pi3,unsigned int*pi4)
-{
-    return p_intf->p_sys->p_mi->requestVideo( p_vout, pi1, pi2, pi3, pi4 );
-}
-static void DoRelease( intf_thread_t *p_intf, void *p_win )
-{
-    return p_intf->p_sys->p_mi->releaseVideo( p_win );
-}
-static int DoControl( intf_thread_t *p_intf, void *p_win, int i_q, va_list a )
-{
-    return p_intf->p_sys->p_mi->controlVideo( p_win, i_q, a );
-}
 
 MainInterface::MainInterface( intf_thread_t *_p_intf ) : QVLCMW( _p_intf )
 {
     /* Variables initialisation */
-    need_components_update = false;
-    bgWidget = NULL; videoWidget = NULL; playlistWidget = NULL;
-    embeddedPlaylistWasActive = videoIsActive = false;
+    // need_components_update = false;
+    bgWidget = NULL;
+    videoWidget = NULL;
+    playlistWidget = NULL;
+    sysTray = NULL;
+    videoIsActive = false;
+    playlistVisible = false;
     input_name = "";
 
+    /* Ask for privacy */
+    askForPrivacy();
+
     /**
      *  Configuration and settings
+     *  Pre-building of interface
      **/
-    settings = new QSettings( "vlc", "vlc-qt-interface" );
-    settings->beginGroup( "MainWindow" );
-
     /* Main settings */
     setFocusPolicy( Qt::StrongFocus );
-    setAcceptDrops(true);
+    setAcceptDrops( true );
     setWindowIcon( QApplication::windowIcon() );
     setWindowOpacity( config_GetFloat( p_intf, "qt-opacity" ) );
 
     /* Set The Video In emebedded Mode or not */
-    videoEmbeddedFlag = false;
-    if( config_GetInt( p_intf, "embedded-video" ) )
-        videoEmbeddedFlag = true;
+    videoEmbeddedFlag = config_GetInt( p_intf, "embedded-video" );
 
-    alwaysVideoFlag = false;
-    if( videoEmbeddedFlag && config_GetInt( p_intf, "qt-always-video" ) )
-        alwaysVideoFlag = true;
+    /* Are we in the enhanced always-video mode or not ? */
+    i_visualmode = config_GetInt( p_intf, "qt-display-mode" );
 
     /* Set the other interface settings */
-    playlistEmbeddedFlag = settings->value( "playlist-embedded", true).toBool();
+    settings = new QSettings( "vlc", "vlc-qt-interface" );
+    settings->beginGroup( "MainWindow" );
+
+    //TODO: I don't like that code
     visualSelectorEnabled = settings->value( "visual-selector", false ).toBool();
-    notificationEnabled = config_GetInt( p_intf, "qt-notification" )
-                          ? true : false;
+    notificationEnabled = (bool)config_GetInt( p_intf, "qt-notification" );
+
     /**************************
      *  UI and Widgets design
      **************************/
     setVLCWindowsTitle();
     handleMainUi( settings );
 
-    /* Menu Bar */
-    QVLCMenu::createMenuBar( this, p_intf, playlistEmbeddedFlag,
-                             visualSelectorEnabled );
+#if 0
+    /* Create a Dock to get the playlist */
+    dockPL = new QDockWidget( qtr( "Playlist" ), this );
+    dockPL->setSizePolicy( QSizePolicy::Preferred,
+                           QSizePolicy::Expanding );
+    dockPL->setFeatures( QDockWidget::AllDockWidgetFeatures );
+    dockPL->setAllowedAreas( Qt::LeftDockWidgetArea
+                           | Qt::RightDockWidgetArea
+                           | Qt::BottomDockWidgetArea );
+    dockPL->hide();
+#endif
 
-    /* Status Bar */
-    /**
-     * TODO: clicking on the elapsed time should switch to the remaining time
-     **/
-    /**
-     * TODO: do we add a label for the current Volume ?
-     **/
-    b_remainingTime = false;
-    timeLabel = new TimeLabel;
-    nameLabel = new QLabel;
-    speedLabel = new QLabel( "1.00x" );
-    timeLabel->setFrameStyle( QFrame::Sunken | QFrame::Panel );
-    speedLabel->setFrameStyle( QFrame::Sunken | QFrame::Panel );
-    statusBar()->addWidget( nameLabel, 8 );
-    statusBar()->addPermanentWidget( speedLabel, 0 );
-    statusBar()->addPermanentWidget( timeLabel, 2 );
-    speedLabel->setContextMenuPolicy ( Qt::CustomContextMenu );
-    timeLabel->setContextMenuPolicy ( Qt::CustomContextMenu );
-    CONNECT( timeLabel, timeLabelClicked(), this, toggleTimeDisplay() );
-    CONNECT( speedLabel, customContextMenuRequested( QPoint ),
-             this, showSpeedMenu( QPoint ) );
-    CONNECT( timeLabel, customContextMenuRequested( QPoint ),
-             this, showTimeMenu( QPoint ) );
+    /************
+     * Menu Bar
+     ************/
+    QVLCMenu::createMenuBar( this, p_intf, visualSelectorEnabled );
 
-    /**********************
-     * Systray Management *
-     **********************/
-    sysTray = NULL;
-    bool b_createSystray = false;
-    bool b_systrayAvailable = QSystemTrayIcon::isSystemTrayAvailable();
-    if( config_GetInt( p_intf, "qt-start-minimized") )
-    {
-        if( b_systrayAvailable ){
-            b_createSystray = true;
-            hide(); //FIXME
-        }
-        else msg_Warn( p_intf, "You can't minize if you haven't a system "
-                "tray bar" );
-    }
-    if( config_GetInt( p_intf, "qt-system-tray") )
-        b_createSystray = true;
+    /* StatusBar Creation */
+    createStatusBar();
 
-    if( b_systrayAvailable && b_createSystray )
-            createSystray();
 
-    /* Init input manager */
+    /********************
+     * Input Manager    *
+     ********************/
     MainInputManager::getInstance( p_intf );
-    ON_TIMEOUT( updateOnTimer() );
-
-    /**
-     * Various CONNECTs
-     **/
 
+    /**************************
+     * Various CONNECTs on IM *
+     **************************/
     /* Connect the input manager to the GUI elements it manages */
+
     /* It is also connected to the control->slider, see the ControlsWidget */
     CONNECT( THEMIM->getIM(), positionUpdated( float, int, int ),
              this, setDisplayPosition( float, int, int ) );
-
+    /* Change the SpeedRate in the Status */
     CONNECT( THEMIM->getIM(), rateChanged( int ), this, setRate( int ) );
 
-    /** Connects on nameChanged() */
+    /**
+     * Connects on nameChanged()
+     * Those connects are not merged because different options can trigger
+     * them down.
+     */
     /* Naming in the controller statusbar */
     CONNECT( THEMIM->getIM(), nameChanged( QString ), this,
              setName( QString ) );
@@ -206,7 +180,9 @@ MainInterface::MainInterface( intf_thread_t *_p_intf ) : QVLCMW( _p_intf )
              setVLCWindowsTitle( QString ) );
     }
 
-    /** CONNECTS on PLAY_STATUS **/
+    /**
+     * CONNECTS on PLAY_STATUS
+     **/
     /* Status on the main controller */
     CONNECT( THEMIM->getIM(), statusChanged( int ), this, setStatus( int ) );
     /* and in the systray */
@@ -216,53 +192,167 @@ MainInterface::MainInterface( intf_thread_t *_p_intf ) : QVLCMW( _p_intf )
                  updateSystrayTooltipStatus( int ) );
     }
 
+    /* END CONNECTS ON IM */
+
+
+    /** OnTimeOut **/
+    /* TODO Remove this function, but so far, there is no choice because there
+       is no intf-should-die variable */
+    ON_TIMEOUT( updateOnTimer() );
+    //ON_TIMEOUT( debug() );
+
     /**
      * Callbacks
      **/
     var_Create( p_intf, "interaction", VLC_VAR_ADDRESS );
     var_AddCallback( p_intf, "interaction", InteractCallback, this );
-    p_intf->b_interaction = VLC_TRUE;
+    p_intf->b_interaction = true;
+
+    var_AddCallback( p_intf->p_libvlc, "intf-show", IntfShowCB, p_intf );
 
     /* Register callback for the intf-popupmenu variable */
-    playlist_t *p_playlist = (playlist_t *)vlc_object_find( p_intf,
-                                        VLC_OBJECT_PLAYLIST, FIND_ANYWHERE );
-    if( p_playlist != NULL )
-    {
-        var_AddCallback( p_playlist, "intf-popupmenu", PopupMenuCB, p_intf );
-        var_AddCallback( p_playlist, "intf-show", IntfShowCB, p_intf );
-        vlc_object_release( p_playlist );
-    }
+    var_AddCallback( p_intf->p_libvlc, "intf-popupmenu", PopupMenuCB, p_intf );
+
+    /* VideoWidget connect mess to avoid different threads speaking to each other */
+    CONNECT( this, askReleaseVideo( void * ),
+             this, releaseVideoSlot( void * ) );
+    if( videoWidget )
+        CONNECT( this, askVideoToResize( unsigned int, unsigned int ),
+                 videoWidget, SetSizing( unsigned int, unsigned int ) );
+
+    CONNECT( this, askUpdate(), this, doComponentsUpdate() );
+
+    CONNECT( controls, advancedControlsToggled( bool ),
+             this, doComponentsUpdate() );
+
+    CONNECT( fullscreenControls, advancedControlsToggled( bool ),
+             this, doComponentsUpdate() );
+
+    /* Size and placement of interface */
+    QVLCTools::restoreWidgetPosition(settings,this,QSize(350,60));
+
+
+    /* Playlist */
+    if( settings->value( "playlist-visible", 0 ).toInt() ) togglePlaylist();
+    settings->endGroup();
+
+    show();
+
+    if( i_visualmode == QT_MINIMAL_MODE )
+        toggleMinimalView();
+
+    /* Update the geometry TODO: is it useful ?*/
+    updateGeometry();
+
+    /*****************************************************
+     * End everything by creating the Systray Management *
+     *****************************************************/
+    initSystray();
 }
 
 MainInterface::~MainInterface()
 {
-    /* Unregister callback for the intf-popupmenu variable */
-    playlist_t *p_playlist = (playlist_t *)vlc_object_find( p_intf,
-                                        VLC_OBJECT_PLAYLIST, FIND_ANYWHERE );
-    if( p_playlist != NULL )
-    {
-        var_DelCallback( p_playlist, "intf-popupmenu", PopupMenuCB, p_intf );
-        var_DelCallback( p_playlist, "intf-show", IntfShowCB, p_intf );
-        vlc_object_release( p_playlist );
-    }
+    msg_Dbg( p_intf, "Destroying the main interface" );
+
+    if( playlistWidget )
+        playlistWidget->savingSettings( settings );
+
+    settings->beginGroup( "MainWindow" );
+
+    // settings->setValue( "playlist-floats", (int)(dockPL->isFloating()) );
+    settings->setValue( "playlist-visible", (int)playlistVisible );
+    settings->setValue( "adv-controls",
+                        getControlsVisibilityStatus() & CONTROLS_ADVANCED );
+
+    if( !videoIsActive )
+        QVLCTools::saveWidgetPosition(settings, this);
+
+    if( bgWidget )
+        settings->setValue( "backgroundSize", bgWidget->size() );
 
-    settings->setValue( "playlist-embedded", playlistEmbeddedFlag );
-    settings->setValue( "adv-controls", getControlsVisibilityStatus() & 0x1 );
-    settings->setValue( "pos", pos() );
     settings->endGroup();
     delete settings;
-    p_intf->b_interaction = VLC_FALSE;
+
+    var_DelCallback( p_intf->p_libvlc, "intf-show", IntfShowCB, p_intf );
+
+    /* Unregister callback for the intf-popupmenu variable */
+    var_DelCallback( p_intf->p_libvlc, "intf-popupmenu", PopupMenuCB, p_intf );
+
+    p_intf->b_interaction = false;
     var_DelCallback( p_intf, "interaction", InteractCallback, this );
 
-    p_intf->pf_request_window = NULL;
-    p_intf->pf_release_window = NULL;
-    p_intf->pf_control_window = NULL;
+    p_intf->p_sys->p_mi = NULL;
 }
 
 /*****************************
  *   Main UI handling        *
  *****************************/
 
+inline void MainInterface::createStatusBar()
+{
+    /****************
+     *  Status Bar  *
+     ****************/
+    /* Widgets Creation*/
+    b_remainingTime = false;
+    timeLabel = new TimeLabel;
+    timeLabel->setText( " --:--/--:-- " );
+    timeLabel->setAlignment( Qt::AlignRight | Qt::AlignVCenter );
+    nameLabel = new QLabel;
+    nameLabel->setTextInteractionFlags( Qt::TextSelectableByMouse
+                                      | Qt::TextSelectableByKeyboard );
+    speedLabel = new SpeedLabel( p_intf, "1.00x" );
+    speedLabel->setContextMenuPolicy ( Qt::CustomContextMenu );
+
+    /* Styling those labels */
+    timeLabel->setFrameStyle( QFrame::Sunken | QFrame::Panel );
+    speedLabel->setFrameStyle( QFrame::Sunken | QFrame::Panel );
+    nameLabel->setFrameStyle( QFrame::Sunken | QFrame::StyledPanel);
+
+    pgBar = new QProgressBar;
+    pgBar->hide();
+
+    /* and adding those */
+    statusBar()->addWidget( nameLabel, 8 );
+    statusBar()->addPermanentWidget( speedLabel, 0 );
+    statusBar()->addPermanentWidget( pgBar, 0 );
+    statusBar()->addPermanentWidget( timeLabel, 0 );
+
+    /* timeLabel behaviour:
+       - double clicking opens the goto time dialog
+       - right-clicking and clicking just toggle between remaining and
+         elapsed time.*/
+    CONNECT( timeLabel, timeLabelClicked(), this, toggleTimeDisplay() );
+    CONNECT( timeLabel, timeLabelDoubleClicked(), THEDP, gotoTimeDialog() );
+    CONNECT( timeLabel, timeLabelDoubleClicked(), this, toggleTimeDisplay() );
+
+    /* Speed Label behaviour:
+       - right click gives the vertical speed slider */
+    CONNECT( speedLabel, customContextMenuRequested( QPoint ),
+             this, showSpeedMenu( QPoint ) );
+}
+
+inline void MainInterface::initSystray()
+{
+    bool b_createSystray = false;
+    bool b_systrayAvailable = QSystemTrayIcon::isSystemTrayAvailable();
+    if( config_GetInt( p_intf, "qt-start-minimized") )
+    {
+        if( b_systrayAvailable )
+        {
+            b_createSystray = true;
+            hide();
+        }
+        else msg_Err( p_intf, "You can't minimize if you haven't a system "
+                "tray bar" );
+    }
+    if( config_GetInt( p_intf, "qt-system-tray") )
+        b_createSystray = true;
+
+    if( b_systrayAvailable && b_createSystray )
+            createSystray();
+}
+
 /**
  * Give the decorations of the Main Window a correct Name.
  * If nothing is given, set it to VLC...
@@ -279,113 +369,214 @@ void MainInterface::setVLCWindowsTitle( QString aTitle )
     }
 }
 
-
 void MainInterface::handleMainUi( QSettings *settings )
 {
     /* Create the main Widget and the mainLayout */
-    QWidget *main = new QWidget( this );
-    mainLayout = new QVBoxLayout( main );
+    QWidget *main = new QWidget;
     setCentralWidget( main );
+    mainLayout = new QVBoxLayout( main );
 
     /* Margins, spacing */
     main->setContentsMargins( 0, 0, 0, 0 );
+    main->setSizePolicy( QSizePolicy::Preferred, QSizePolicy::Maximum );
+    mainLayout->setSpacing( 0 );
     mainLayout->setMargin( 0 );
 
     /* Create the CONTROLS Widget */
-    controls = new ControlsWidget( p_intf,
-                   settings->value( "adv-controls", false ).toBool() );
+    bool b_shiny = config_GetInt( p_intf, "qt-blingbling" );
+    controls = new ControlsWidget( p_intf, this,
+                   settings->value( "adv-controls", false ).toBool(),
+                   b_shiny );
 
-    /* Configure the Controls */
-    BUTTON_SET_IMG( controls->playlistButton, "" , playlist_icon.png,
-                    playlistEmbeddedFlag ?  qtr( "Show playlist" ) :
-                                            qtr( "Open playlist" ) );
-    BUTTONACT( controls->playlistButton, togglePlaylist() );
+    /* Create the FULLSCREEN CONTROLS Widget */
+    /* bool b_shiny = config_GetInt( p_intf, "qt-blingbling" ); */
+    fullscreenControls = new FullscreenControllerWidget( p_intf, this,
+                   settings->value( "adv-controls", false ).toBool(),
+                   b_shiny );
 
     /* Add the controls Widget to the main Widget */
-    mainLayout->addWidget( controls );
+    mainLayout->insertWidget( 0, controls, 0, Qt::AlignBottom );
 
     /* Create the Speed Control Widget */
     speedControl = new SpeedControlWidget( p_intf );
     speedControlMenu = new QMenu( this );
-    QWidgetAction *widgetAction = new QWidgetAction( this );
+
+    QWidgetAction *widgetAction = new QWidgetAction( speedControl );
     widgetAction->setDefaultWidget( speedControl );
     speedControlMenu->addAction( widgetAction );
 
-    /* Set initial size */
-    resize( PREF_W, PREF_H );
-    addSize = QSize( mainLayout->margin() * 2, PREF_H );
-
     /* Visualisation */
+    /* Disabled for now, they SUCK */
+    #if 0
     visualSelector = new VisualSelector( p_intf );
     mainLayout->insertWidget( 0, visualSelector );
     visualSelector->hide();
+    #endif
 
     /* And video Outputs */
-    if( alwaysVideoFlag )
+    if( i_visualmode == QT_ALWAYS_VIDEO_MODE ||
+        i_visualmode == QT_MINIMAL_MODE )
     {
         bgWidget = new BackgroundWidget( p_intf );
-        bgWidget->widgetSize = settings->value( "backgroundSize",
-                                           QSize( 300, 300 ) ).toSize();
-        bgWidget->resize( bgWidget->widgetSize );
+        bgWidget->resize(
+             settings->value( "backgroundSize", QSize( 300, 150 ) ).toSize() );
         bgWidget->updateGeometry();
         mainLayout->insertWidget( 0, bgWidget );
+        CONNECT( this, askBgWidgetToToggle(), bgWidget, toggle() );
     }
 
     if( videoEmbeddedFlag )
     {
         videoWidget = new VideoWidget( p_intf );
-        videoWidget->widgetSize = QSize( 1, 1 );
-        //videoWidget->resize( videoWidget->widgetSize );
-        mainLayout->insertWidget( 0, videoWidget );
-
-        p_intf->pf_request_window  = ::DoRequest;
-        p_intf->pf_release_window  = ::DoRelease;
-        p_intf->pf_control_window  = ::DoControl;
+        mainLayout->insertWidget( 0, videoWidget, 10 );
     }
 
     /* Finish the sizing */
-    setMinimumSize( PREF_W, addSize.height() );
+    main->updateGeometry();
+}
+
+inline void MainInterface::askForPrivacy()
+{
+    /**
+     * Ask for the network policy on FIRST STARTUP
+     **/
+    if( config_GetInt( p_intf, "qt-privacy-ask") )
+    {
+        QList<ConfigControl *> controls;
+        if( privacyDialog( controls ) == QDialog::Accepted )
+        {
+            QList<ConfigControl *>::Iterator i;
+            for(  i = controls.begin() ; i != controls.end() ; i++ )
+            {
+                ConfigControl *c = qobject_cast<ConfigControl *>(*i);
+                c->doApply( p_intf );
+            }
+
+            config_PutInt( p_intf,  "qt-privacy-ask" , 0 );
+            /* We have to save here because the user may not launch Prefs */
+            config_SaveConfigFile( p_intf, NULL );
+        }
+    }
 }
 
+int MainInterface::privacyDialog( QList<ConfigControl *> controls )
+{
+    QDialog *privacy = new QDialog();
+
+    privacy->setWindowTitle( qtr( "Privacy and Network Policies" ) );
+
+    QGridLayout *gLayout = new QGridLayout( privacy );
+
+    QGroupBox *blabla = new QGroupBox( qtr( "Privacy and Network Warning" ) );
+    QGridLayout *blablaLayout = new QGridLayout( blabla );
+    QLabel *text = new QLabel( qtr(
+        "<p>The <i>VideoLAN Team</i> doesn't like when an application goes "
+        "online without authorization.</p>\n "
+        "<p><i>VLC media player</i> can request limited information on "
+        "the Internet, especially to get CD covers or to know "
+        "if updates are available.</p>\n"
+        "<p><i>VLC media player</i> <b>DOES NOT</b> send or collect <b>ANY</b> "
+        "information, even anonymously, about your usage.</p>\n"
+        "<p>Therefore please check the following options, the default being "
+        "almost no access on the web.</p>\n") );
+    text->setWordWrap( true );
+    text->setTextFormat( Qt::RichText );
+
+    blablaLayout->addWidget( text, 0, 0 ) ;
+
+    QGroupBox *options = new QGroupBox;
+    QGridLayout *optionsLayout = new QGridLayout( options );
+
+    gLayout->addWidget( blabla, 0, 0, 1, 3 );
+    gLayout->addWidget( options, 1, 0, 1, 3 );
+    module_config_t *p_config;
+    ConfigControl *control;
+    int line = 0;
+
+#define CONFIG_GENERIC( option, type )                            \
+    p_config =  config_FindConfig( VLC_OBJECT(p_intf), option );  \
+    if( p_config )                                                \
+    {                                                             \
+        control =  new type ## ConfigControl( VLC_OBJECT(p_intf), \
+                p_config, options, false, optionsLayout, line );  \
+        controls.append( control );                               \
+    }
+
+#define CONFIG_GENERIC_NOBOOL( option, type )                     \
+    p_config =  config_FindConfig( VLC_OBJECT(p_intf), option );  \
+    if( p_config )                                                \
+    {                                                             \
+        control =  new type ## ConfigControl( VLC_OBJECT(p_intf), \
+                p_config, options, optionsLayout, line );  \
+        controls.append( control );                               \
+    }
+
+    CONFIG_GENERIC( "album-art", IntegerList ); line++;
+#ifdef UPDATE_CHECK
+    CONFIG_GENERIC_NOBOOL( "qt-updates-notif", Bool ); line++;
+    CONFIG_GENERIC_NOBOOL( "qt-updates-days", Integer ); line++;
+#endif
+
+    QPushButton *ok = new QPushButton( qtr( "OK" ) );
+
+    gLayout->addWidget( ok, 2, 2 );
+
+    CONNECT( ok, clicked(), privacy, accept() );
+    return privacy->exec();
+}
+
+
 /**********************************************************************
  * Handling of sizing of the components
  **********************************************************************/
-void MainInterface::calculateInterfaceSize()
+
+/* This function is probably wrong, but we don't have many many choices...
+   Since we can't know from the playlist Widget if we are inside a dock or not,
+   because the playlist Widget can be called by THEDP, as a separate windows for
+   the skins.
+   Maybe the other solution is to redefine the sizeHint() of the playlist and
+   ask _parent->isFloating()...
+   If you think this would be better, please FIXME it...
+*/
+#if 0
+QSize MainInterface::sizeHint() const
 {
-    int width = 0, height = 0;
+    int nwidth  = controls->sizeHint().width();
+    int nheight = controls->isVisible() ?
+                  controls->size().height()
+                  + menuBar()->size().height()
+                  + statusBar()->size().height()
+                  : 0 ;
+
+    msg_Dbg( p_intf, "1 %i %i", nheight, nwidth );
     if( VISIBLE( bgWidget ) )
     {
-        width = bgWidget->widgetSize.width();
-        height = bgWidget->widgetSize.height();
-        assert( !(playlistWidget && playlistWidget->isVisible() ) );
-    }
-    else if( VISIBLE( playlistWidget ) )
-    {
-        width = playlistWidget->widgetSize.width();
-        height = playlistWidget->widgetSize.height();
+        nheight += bgWidget->size().height();
+        nwidth  = bgWidget->size().width();
+        msg_Dbg( p_intf, "1b %i %i", nheight, nwidth );
     }
     else if( videoIsActive )
     {
-        width =  videoWidget->widgetSize.width() ;
-        height = videoWidget->widgetSize.height();
-    }
-    else
-    {
-        width = PREF_W - addSize.width();
-        height = PREF_H - addSize.height();
+        nheight += videoWidget->size().height();
+        nwidth  = videoWidget->size().width();
+        msg_Dbg( p_intf, "2 %i %i", nheight, nwidth );
     }
-    if( VISIBLE( visualSelector ) )
-        height += visualSelector->height();
-/*    if( VISIBLE( advControls) )
+/*    if( !dockPL->isFloating() && dockPL->isVisible() && dockPL->widget()  )
     {
-        height += advControls->sizeHint().height();
+        nheight += dockPL->size().height();
+        nwidth = __MAX( nwidth, dockPL->size().width() );
+        msg_Dbg( p_intf, "3 %i %i", nheight, nwidth );
     }*/
-    mainSize = QSize( width + addSize.width(), height + addSize.height() );
+    msg_Dbg( p_intf, "4 %i %i", nheight, nwidth );
+    return QSize( nwidth, nheight );
 }
-
+#endif
+#if 0
+/* FIXME This is dead code and need to be removed AT THE END */
 void MainInterface::resizeEvent( QResizeEvent *e )
 {
-    videoWidget->widgetSize.setWidth(  e->size().width() - addSize.width() );
+    if( videoWidget )
+        videoWidget->widgetSize.setWidth( e->size().width() - addSize.width() );
     if( videoWidget && videoIsActive && videoWidget->widgetSize.height() > 1 )
     {
         SET_WH( videoWidget, e->size().width() - addSize.width(),
@@ -394,50 +585,83 @@ void MainInterface::resizeEvent( QResizeEvent *e )
     }
     if( VISIBLE( playlistWidget ) )
     {
-        SET_WH( playlistWidget , e->size().width() - addSize.width(),
-                                 e->size().height() - addSize.height() );
+//        SET_WH( playlistWidget , e->size().width() - addSize.width(),
+              //                   e->size().height() - addSize.height() );
         playlistWidget->updateGeometry();
     }
 }
+#endif
 
-/****************************************************************************
- * Small right-click menus
- ****************************************************************************/
-void MainInterface::showSpeedMenu( QPoint pos )
+void MainInterface::requestLayoutUpdate()
+{
+    emit askUpdate();
+}
+
+//FIXME remove me at the end...
+void MainInterface::debug()
 {
-    speedControlMenu->exec( QCursor::pos() - pos + QPoint( 0, speedLabel->height() ) );
+    msg_Dbg( p_intf, "size: %i - %i", size().height(), size().width() );
+    msg_Dbg( p_intf, "sizeHint: %i - %i", sizeHint().height(), sizeHint().width() );
+    if( videoWidget && videoWidget->isVisible() )
+    {
+//    sleep( 10 );
+    msg_Dbg( p_intf, "size: %i - %i", size().height(), size().width() );
+    msg_Dbg( p_intf, "sizeHint: %i - %i", sizeHint().height(), sizeHint().width() );
+    }
+    adjustSize();
 }
 
-void MainInterface::showTimeMenu( QPoint pos )
+/****************************************************************************
+ * Small right-click menu for rate control
+ ****************************************************************************/
+void MainInterface::showSpeedMenu( QPoint pos )
 {
-    QMenu menu( this );
-    menu.addAction(  qtr("Elapsed Time") , this, SLOT( setElapsedTime() ) );
-    menu.addAction(  qtr("Remaining Time") , this, SLOT( setRemainTime() ) );
-    menu.exec( QCursor::pos() - pos +QPoint( 0, timeLabel->height() ) );
+    speedControlMenu->exec( QCursor::pos() - pos
+                          + QPoint( 0, speedLabel->height() ) );
 }
 
 /****************************************************************************
  * Video Handling
  ****************************************************************************/
+class SetVideoOnTopQtEvent : public QEvent
+{
+public:
+    SetVideoOnTopQtEvent( bool _onTop ) :
+      QEvent( (QEvent::Type)SetVideoOnTopEvent_Type ), onTop( _onTop)
+    {}
+
+    bool OnTop() const
+    {
+        return onTop;
+    }
+
+private:
+    bool onTop;
+};
+
+/**
+ * README
+ * README
+ * Thou shall not call/resize/hide widgets from on another thread.
+ * This is wrong, and this is TEH reason to emit signals on those Video Functions
+ **/
 void *MainInterface::requestVideo( vout_thread_t *p_nvout, int *pi_x,
                                    int *pi_y, unsigned int *pi_width,
                                    unsigned int *pi_height )
 {
+    bool bgWasVisible = false;
+
+    /* Request the videoWidget */
     void *ret = videoWidget->request( p_nvout,pi_x, pi_y, pi_width, pi_height );
-    if( ret )
+    if( ret ) /* The videoWidget is available */
     {
-        videoIsActive = true;
-        if( VISIBLE( playlistWidget ) )
-        {
-            embeddedPlaylistWasActive = true;
-//            playlistWidget->hide();
-        }
-        bool bgWasVisible = false;
+        /* Did we have a bg ? Hide it! */
         if( VISIBLE( bgWidget) )
         {
             bgWasVisible = true;
-            bgWidget->hide();
+            emit askBgWidgetToToggle();
         }
+#if 0
         if( THEMIM->getIM()->hasVideo() || !bgWasVisible )
         {
             videoWidget->widgetSize = QSize( *pi_width, *pi_height );
@@ -449,76 +673,64 @@ void *MainInterface::requestVideo( vout_thread_t *p_nvout, int *pi_x,
             // videoWidget->widgetSize = bgWidget->widgeTSize;
             videoWidget->widgetSize = QSize( *pi_width, *pi_height );
         }
-        videoWidget->updateGeometry(); // Needed for deinterlace
-        need_components_update = true;
+#endif
+        videoIsActive = true;
+
+//        emit askVideoToResize( *pi_width, *pi_height );
+        emit askUpdate();
+
+        fullscreenControls->attachVout( p_nvout );
     }
     return ret;
 }
 
-void MainInterface::releaseVideo( void *p_win )
+void MainInterface::releaseVideo( vout_thread_t *p_vout, void *p_win )
+{
+    fullscreenControls->detachVout( p_vout );
+    emit askReleaseVideo( p_win );
+}
+
+void MainInterface::releaseVideoSlot( void *p_win )
 {
     videoWidget->release( p_win );
-    videoWidget->widgetSize = QSize( 0, 0 );
-    videoWidget->resize( videoWidget->widgetSize );
+    videoWidget->hide();
 
-    if( embeddedPlaylistWasActive )
-        playlistWidget->show();
-    else if( bgWidget )
+    if( bgWidget )// WRONG
         bgWidget->show();
 
+    adjustSize();
     videoIsActive = false;
-    need_components_update = true;
 }
 
-class SetVideoOnTopQtEvent : public QEvent
-{
-public:
-    SetVideoOnTopQtEvent( bool _onTop ) :
-      QEvent( (QEvent::Type)SetVideoOnTopEvent_Type ), onTop( _onTop)
-    {
-    }
-
-    bool OnTop() const
-    {
-        return onTop;
-    }
-
-private:
-    bool onTop;
-};
-
 int MainInterface::controlVideo( void *p_window, int i_query, va_list args )
 {
-    int i_ret = VLC_EGENERIC;
+    int i_ret = VLC_SUCCESS;
     switch( i_query )
     {
         case VOUT_GET_SIZE:
         {
             unsigned int *pi_width  = va_arg( args, unsigned int * );
             unsigned int *pi_height = va_arg( args, unsigned int * );
-            *pi_width = videoWidget->widgetSize.width();
-            *pi_height = videoWidget->widgetSize.height();
-            i_ret = VLC_SUCCESS;
+            *pi_width = videoWidget->videoSize.width();
+            *pi_height = videoWidget->videoSize.height();
             break;
         }
         case VOUT_SET_SIZE:
         {
             unsigned int i_width  = va_arg( args, unsigned int );
             unsigned int i_height = va_arg( args, unsigned int );
-            videoWidget->widgetSize = QSize( i_width, i_height );
-            // videoWidget->updateGeometry();
-            need_components_update = true;
-            i_ret = VLC_SUCCESS;
+            emit askVideoToResize( i_width, i_height );
+            emit askUpdate();
             break;
         }
         case VOUT_SET_STAY_ON_TOP:
         {
             int i_arg = va_arg( args, int );
             QApplication::postEvent( this, new SetVideoOnTopQtEvent( i_arg ) );
-            i_ret = VLC_SUCCESS;
             break;
         }
         default:
+            i_ret = VLC_EGENERIC;
             msg_Warn( p_intf, "unsupported control query" );
             break;
     }
@@ -533,82 +745,85 @@ int MainInterface::controlVideo( void *p_window, int i_query, va_list args )
  **/
 void MainInterface::togglePlaylist()
 {
-    // Toggle the playlist dialog if not embedded and return
-    if( !playlistEmbeddedFlag )
-    {
-        if( playlistWidget )
-        {
-            /// \todo Destroy it
-        }
-        THEDP->playlistDialog();
-        return;
-    }
-
-    // Create the playlist Widget and destroy the existing dialog
+    THEDP->playlistDialog();
+#if 0
+    /* CREATION
+    If no playlist exist, then create one and attach it to the DockPL*/
     if( !playlistWidget )
     {
-        PlaylistDialog::killInstance();
-        playlistWidget = new PlaylistWidget( p_intf );
-        mainLayout->insertWidget( 0, playlistWidget );
-        playlistWidget->widgetSize = settings->value( "playlistSize",
-                                               QSize( 650, 310 ) ).toSize();
-        playlistWidget->hide();
-        if(bgWidget)
-        CONNECT( playlistWidget, artSet( QString ), bgWidget, setArt(QString) );
-    }
+        playlistWidget = new PlaylistWidget( p_intf, settings, dockPL );
 
-    // And toggle visibility
-    if( VISIBLE( playlistWidget ) )
-    {
-        playlistWidget->hide();
-        if( bgWidget ) bgWidget->show();
-        if( videoIsActive )
+        /* Add it to the parent DockWidget */
+        dockPL->setWidget( playlistWidget );
+
+        /* Add the dock to the main Interface */
+        addDockWidget( Qt::BottomDockWidgetArea, dockPL );
+
+        /* Make the playlist floating is requested. Default is not. */
+        settings->beginGroup( "MainWindow" );
+        if( settings->value( "playlist-floats", 1 ).toInt() )
         {
-            videoWidget->widgetSize = savedVideoSize;
-            videoWidget->resize( videoWidget->widgetSize );
-            videoWidget->updateGeometry();
-            if( bgWidget ) bgWidget->hide();
+            msg_Dbg( p_intf, "we don't want the playlist inside");
+            dockPL->setFloating( true );
         }
+        settings->endGroup();
+        settings->beginGroup( "playlist" );
+        dockPL->move( settings->value( "pos", QPoint( 0,0 ) ).toPoint() );
+        QSize newSize = settings->value( "size", QSize( 400, 300 ) ).toSize();
+        if( newSize.isValid() )
+            dockPL->resize( newSize );
+        settings->endGroup();
+
+        dockPL->show();
+        playlistVisible = true;
     }
     else
     {
-        playlistWidget->show();
-        if( videoIsActive )
-        {
-            savedVideoSize = videoWidget->widgetSize;
-            videoWidget->widgetSize.setHeight( 0 );
-            videoWidget->resize( videoWidget->widgetSize );
-            videoWidget->updateGeometry();
-        }
-        if( VISIBLE( bgWidget ) ) bgWidget->hide();
+    /* toggle the visibility of the playlist */
+       TOGGLEV( dockPL );
+       resize( sizeHint() );
+       playlistVisible = !playlistVisible;
     }
-
-    doComponentsUpdate();
+    #endif
 }
 
+/* Function called from the menu to undock the playlist */
 void MainInterface::undockPlaylist()
 {
-    if( playlistWidget )
-    {
-        playlistWidget->hide();
-        playlistWidget->deleteLater();
-        mainLayout->removeWidget( playlistWidget );
-        playlistWidget = NULL;
-        playlistEmbeddedFlag = false;
+//    dockPL->setFloating( true );
+    adjustSize();
+}
 
-        menuBar()->clear();
-        QVLCMenu::createMenuBar( this, p_intf, false, visualSelectorEnabled);
+void MainInterface::toggleMinimalView()
+{
+    /* HACK for minimalView, see menus.cpp */
+    if( !menuBar()->isVisible() ) QVLCMenu::minimalViewAction->toggle();
+    TOGGLEV( menuBar() );
+    TOGGLEV( controls );
+    TOGGLEV( statusBar() );
+    doComponentsUpdate();
+}
 
-        if( videoIsActive )
-        {
-            videoWidget->widgetSize = savedVideoSize;
-            videoWidget->resize( videoWidget->widgetSize );
-            videoWidget->updateGeometry();
-        }
+/* Video widget cannot do this synchronously as it runs in another thread */
+/* Well, could it, actually ? Probably dangerous ... */
+void MainInterface::doComponentsUpdate()
+{
+    msg_Dbg( p_intf, "Updating the geometry" );
+//    resize( sizeHint() );
+    debug();
+}
 
-        doComponentsUpdate();
-        THEDP->playlistDialog();
-    }
+/* toggling advanced controls buttons */
+void MainInterface::toggleAdvanced()
+{
+    controls->toggleAdvanced();
+}
+
+/* Get the visibility status of the controls (hidden or not, advanced or not) */
+int MainInterface::getControlsVisibilityStatus()
+{
+    return( (controls->isVisible() ? CONTROLS_VISIBLE : CONTROLS_HIDDEN )
+                + CONTROLS_ADVANCED * controls->b_advancedVisible );
 }
 
 #if 0
@@ -633,32 +848,6 @@ void MainInterface::visual()
 }
 #endif
 
-void MainInterface::toggleMenus()
-{
-    TOGGLEV( controls );
-    TOGGLEV( statusBar() );
-    updateGeometry();
-}
-
-/* Video widget cannot do this synchronously as it runs in another thread */
-/* Well, could it, actually ? Probably dangerous ... */
-void MainInterface::doComponentsUpdate()
-{
-    calculateInterfaceSize();
-    resize( mainSize );
-}
-
-void MainInterface::toggleAdvanced()
-{
-    controls->toggleAdvanced();
-}
-
-int MainInterface::getControlsVisibilityStatus()
-{
-    return( (controls->isVisible() ? 0x2 : 0x0 )
-                + controls->b_advancedVisible );
-}
-
 /************************************************************************
  * Other stuff
  ************************************************************************/
@@ -666,57 +855,66 @@ void MainInterface::setDisplayPosition( float pos, int time, int length )
 {
     char psz_length[MSTRTIME_MAX_SIZE], psz_time[MSTRTIME_MAX_SIZE];
     secstotimestr( psz_length, length );
-    secstotimestr( psz_time, b_remainingTime ? length - time : time );
-    QString title; title.sprintf( "%s/%s", psz_time, psz_length );
-    if( b_remainingTime ) timeLabel->setText( " -"+title+" " );
-    else timeLabel->setText( " "+title+" " );
+    secstotimestr( psz_time, ( b_remainingTime && length ) ? length - time
+                                                           : time );
+
+    QString timestr;
+    timestr.sprintf( "%s/%s", psz_time,
+                            ( !length && time ) ? "--:--" : psz_length );
+
+    /* Add a minus to remaining time*/
+    if( b_remainingTime && length ) timeLabel->setText( " -"+timestr+" " );
+    else timeLabel->setText( " "+timestr+" " );
 }
 
 void MainInterface::toggleTimeDisplay()
 {
-    b_remainingTime = ( b_remainingTime ? false : true );
+    b_remainingTime = !b_remainingTime;
 }
 
-void MainInterface::setElapsedTime(){ b_remainingTime = false; }
-void MainInterface::setRemainTime(){ b_remainingTime = true; }
-
 void MainInterface::setName( QString name )
 {
-    input_name = name;
-    nameLabel->setText( " " + name+" " );
+    input_name = name; /* store it for the QSystray use */
+    /* Display it in the status bar, but also as a Tooltip in case it doesn't
+       fit in the label */
+    nameLabel->setText( " " + name + " " );
+    nameLabel->setToolTip( " " + name +" " );
 }
 
 void MainInterface::setStatus( int status )
 {
+    msg_Dbg( p_intf, "I was here, updating your status" );
+    /* Forward the status to the controls to toggle Play/Pause */
     controls->setStatus( status );
+    fullscreenControls->setStatus( status );
+
+    controls->updateInput();
+    fullscreenControls->updateInput();
+    speedControl->setEnable( THEMIM->getIM()->hasInput() );
+
+    /* And in the systray for the menu */
     if( sysTray )
-        updateSystrayMenu( status );
+        QVLCMenu::updateSystrayMenu( this, p_intf );
 }
 
 void MainInterface::setRate( int rate )
 {
     QString str;
-    str.setNum( ( 1000 / (double)rate), 'f', 2 );
+    str.setNum( ( 1000 / (double)rate ), 'f', 2 );
     str.append( "x" );
     speedLabel->setText( str );
+    speedLabel->setToolTip( str );
     speedControl->updateControls( rate );
 }
 
 void MainInterface::updateOnTimer()
 {
-    /* \todo Make this event-driven */
+    /* No event for dying */
     if( intf_ShouldDie( p_intf ) )
     {
         QApplication::closeAllWindows();
         QApplication::quit();
     }
-    if( need_components_update )
-    {
-        doComponentsUpdate();
-        need_components_update = false;
-    }
-
-    controls->updateOnTimer();
 }
 
 /*****************************************************************************
@@ -729,7 +927,11 @@ void MainInterface::updateOnTimer()
  **/
 void MainInterface::createSystray()
 {
-    QIcon iconVLC =  QIcon( QPixmap( ":/vlc128.png" ) );
+    QIcon iconVLC;
+    if( QDate::currentDate().dayOfYear() >= 354 )
+        iconVLC =  QIcon( QPixmap( ":/vlc128-christmas.png" ) );
+    else
+        iconVLC =  QIcon( QPixmap( ":/vlc128.png" ) );
     sysTray = new QSystemTrayIcon( iconVLC, this );
     sysTray->setToolTip( qtr( "VLC media player" ));
 
@@ -743,21 +945,12 @@ void MainInterface::createSystray()
             this, handleSystrayClick( QSystemTrayIcon::ActivationReason ) );
 }
 
-/**
- * Update the menu of the Systray Icon.
- * May be unneedded, since it just calls QVLCMenu::update
- * FIXME !!!
- **/
-void MainInterface::updateSystrayMenu( int status )
-{
-    QVLCMenu::updateSystrayMenu( this, p_intf ) ;
-}
-
 /**
  * Updates the Systray Icon's menu and toggle the main interface
  */
 void MainInterface::toggleUpdateSystrayMenu()
 {
+    /* If hidden, show it */
     if( isHidden() )
     {
         show();
@@ -765,11 +958,13 @@ void MainInterface::toggleUpdateSystrayMenu()
     }
     else if( isMinimized() )
     {
+        /* Minimized */
         showNormal();
         activateWindow();
     }
     else
     {
+        /* Visible */
 #ifdef WIN32
         /* check if any visible window is above vlc in the z-order,
          * but ignore the ones always on top */
@@ -806,7 +1001,7 @@ void MainInterface::handleSystrayClick(
         case QSystemTrayIcon::MiddleClick:
             sysTray->showMessage( qtr( "VLC media player" ),
                     qtr( "Control menu for the player" ),
-                    QSystemTrayIcon::Information, 4000 );
+                    QSystemTrayIcon::Information, 3000 );
             break;
     }
 }
@@ -814,7 +1009,6 @@ void MainInterface::handleSystrayClick(
 /**
  * Updates the name of the systray Icon tooltip.
  * Doesn't check if the systray exists, check before you call it.
- * FIXME !!! Fusion with next function ?
  **/
 void MainInterface::updateSystrayTooltipName( QString name )
 {
@@ -828,7 +1022,7 @@ void MainInterface::updateSystrayTooltipName( QString name )
         if( notificationEnabled && ( isHidden() || isMinimized() ) )
         {
             sysTray->showMessage( qtr( "VLC media player" ), name,
-                    QSystemTrayIcon::NoIcon, 4000 );
+                    QSystemTrayIcon::NoIcon, 3000 );
         }
     }
 }
@@ -842,6 +1036,7 @@ void MainInterface::updateSystrayTooltipStatus( int i_status )
     switch( i_status )
     {
         case  0:
+        case  END_S:
             {
                 sysTray->setToolTip( qtr( "VLC media player" ) );
                 break;
@@ -849,7 +1044,6 @@ void MainInterface::updateSystrayTooltipStatus( int i_status )
         case PLAYING_S:
             {
                 sysTray->setToolTip( input_name );
-                //+ " - " + qtr( "Playing" ) );
                 break;
             }
         case PAUSE_S:
@@ -875,7 +1069,7 @@ void MainInterface::dropEvent(QDropEvent *event)
         {
             if( input_AddSubtitles( THEMIM->getInput(),
                                     qtu( mimeData->urls()[0].toString() ),
-                                    VLC_TRUE ) )
+                                    true ) )
             {
                 event->acceptProposedAction();
                 return;
@@ -883,12 +1077,13 @@ void MainInterface::dropEvent(QDropEvent *event)
         }
      }
      bool first = true;
-     foreach( QUrl url, mimeData->urls() ) {
-        QString s = url.toString();
+     foreach( QUrl url, mimeData->urls() )
+     {
+        QString s = url.toLocalFile();
         if( s.length() > 0 ) {
             playlist_Add( THEPL, qtu(s), NULL,
                           PLAYLIST_APPEND | (first ? PLAYLIST_GO:0),
-                          PLAYLIST_END, VLC_TRUE, VLC_FALSE );
+                          PLAYLIST_END, true, false );
             first = false;
         }
      }
@@ -912,6 +1107,7 @@ void MainInterface::dragLeaveEvent(QDragLeaveEvent *event)
  ************************************************************************/
 void MainInterface::customEvent( QEvent *event )
 {
+#if 0
     if( event->type() == PLDockEvent_Type )
     {
         PlaylistDialog::killInstance();
@@ -920,7 +1116,9 @@ void MainInterface::customEvent( QEvent *event )
         QVLCMenu::createMenuBar(this, p_intf, true, visualSelectorEnabled);
         togglePlaylist();
     }
-    else if ( event->type() == SetVideoOnTopEvent_Type )
+#endif
+    /*else */
+    if ( event->type() == SetVideoOnTopEvent_Type )
     {
         SetVideoOnTopQtEvent* p_event = (SetVideoOnTopQtEvent*)event;
         if( p_event->OnTop() )
@@ -933,8 +1131,15 @@ void MainInterface::customEvent( QEvent *event )
 
 void MainInterface::keyPressEvent( QKeyEvent *e )
 {
+    if( ( e->modifiers() &  Qt::ControlModifier ) && ( e->key() & Qt::Key_H )
+          && menuBar()->isHidden() )
+    {
+        toggleMinimalView();
+        e->accept();
+    }
+
     int i_vlck = qtEventToVLCKey( e );
-    if( i_vlck >= 0 )
+    if( i_vlck > 0 )
     {
         var_SetInteger( p_intf->p_libvlc, "key-pressed", i_vlck );
         e->accept();
@@ -953,7 +1158,15 @@ void MainInterface::wheelEvent( QWheelEvent *e )
 void MainInterface::closeEvent( QCloseEvent *e )
 {
     hide();
-    vlc_object_kill( p_intf );
+    THEDP->quit();
+}
+
+void MainInterface::toggleFullScreen( void )
+{
+    if( isFullScreen() )
+        showNormal();
+    else
+        showFullScreen();
 }
 
 /*****************************************************************************
@@ -990,13 +1203,13 @@ static int PopupMenuCB( vlc_object_t *p_this, const char *psz_variable,
 }
 
 /*****************************************************************************
- * IntfShowCB: callback triggered by the intf-show playlist variable.
+ * IntfShowCB: callback triggered by the intf-show libvlc variable.
  *****************************************************************************/
 static int IntfShowCB( vlc_object_t *p_this, const char *psz_variable,
                        vlc_value_t old_val, vlc_value_t new_val, void *param )
 {
     intf_thread_t *p_intf = (intf_thread_t *)param;
-    //p_intf->p_sys->b_intf_show = VLC_TRUE;
+    p_intf->p_sys->p_mi->requestLayoutUpdate();
 
     return VLC_SUCCESS;
 }