[vlc-commits] Qt4: Fix a small playlist docking bug

Edward Wang git at videolan.org
Tue Jan 3 09:16:55 CET 2012


vlc | branch: master | Edward Wang <edward.c.wang at compdigitec.com> | Mon Jan  2 23:57:36 2012 -0500| [6a0131dae6a119a72fbec2bce995ca1dfcd9b5be] | committer: Jean-Baptiste Kempf

Qt4: Fix a small playlist docking bug

Fix a small playlist docking bug where the playlistWidget is not properly set as a Qt::Widget, causing it to behave inappropriately.

Close #5773

Signed-off-by: Jean-Baptiste Kempf <jb at videolan.org>

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=6a0131dae6a119a72fbec2bce995ca1dfcd9b5be
---

 modules/gui/qt4/main_interface.cpp |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/modules/gui/qt4/main_interface.cpp b/modules/gui/qt4/main_interface.cpp
index f0323d5..2ac2397 100644
--- a/modules/gui/qt4/main_interface.cpp
+++ b/modules/gui/qt4/main_interface.cpp
@@ -856,13 +856,13 @@ void MainInterface::dockPlaylist( bool p_docked )
     }
     else /* Previously undocked */
     {
-        /* If playlist is invisible don't show it */
-        if( !playlistWidget->isVisible() ) return;
-
         QVLCTools::saveWidgetPosition( p_intf, "Playlist", playlistWidget );
         playlistWidget->setWindowFlags( Qt::Widget ); // Probably a Qt bug here
         // It would be logical that QStackWidget::addWidget reset the flags...
         stackCentralW->addWidget( playlistWidget );
+
+        /* If playlist is invisible don't show it */
+        if( !playlistWidget->isVisible() ) return;
         showTab( playlistWidget );
     }
     playlistVisible = true;



More information about the vlc-commits mailing list