Bug 86884 - Universal SideBar flickers too much
Summary: Universal SideBar flickers too much
Status: RESOLVED FIXED
Alias: None
Product: kicker
Classification: Plasma
Component: sidebar (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Joseph Wenninger
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-08-10 06:58 UTC by Gustavo Sverzut Barbieri
Modified: 2004-12-05 00:47 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
Make sidebar to start closed (791 bytes, patch)
2004-08-10 07:00 UTC, Gustavo Sverzut Barbieri
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Gustavo Sverzut Barbieri 2004-08-10 06:58:53 UTC
Version:            (using KDE Devel)
Installed from:    Compiled sources

Universal SideBar flickers too much, that's ugly, annoying and makes everything looks slower than it really is.

Thing that may help:
   - First add contents (layouts and widgets) to it before expand, not after. Now it looks like the sidebar is expanded, the display is refreshed, then it adds the contents to be shown. It should do the other way.
   - When collapsing a tab name, hold update before everything is done. Right now it make the icon smaller (remove the text) and refresh, then move everything up and then refresh again... The intermediary refresh should be eliminated.
   - Start the sidebar closed. When kicker is loaded, the sidebar is opened then it's closed, this make everything flicker. If the sidebar is not in auto-hide mode, it need to shift windows and everything flickers even more.
Comment 1 Gustavo Sverzut Barbieri 2004-08-10 07:00:49 UTC
Created attachment 7057 [details]
Make sidebar to start closed

This makes universal sidebar to start closed, avoid initial flickering and
jumping.
Comment 2 Gustavo Sverzut Barbieri 2004-08-10 08:00:51 UTC
Seems that the two first items are due poor implementation of SideBar_Widget.

Since I'm no QT/KDE expert and I don't understand the internal logic very well, if this is true, please could you contact the right people to get this fixed?
Comment 3 Gustavo Sverzut Barbieri 2004-08-24 23:54:28 UTC
Ok, 3.3 released, could someone commit the patch?
Comment 4 Gustavo Sverzut Barbieri 2004-10-29 01:20:50 UTC
I still don't see it in 3.3.1, will it be in for 3.3.2?
Comment 5 Aaron J. Seigo 2004-12-05 00:47:28 UTC
CVS commit by aseigo: 

don't flicker up at start up
BUG:86884


  M +1 -1      sidebarextension.cpp   1.10


--- kdebase/kicker/extensions/sidebar/sidebarextension.cpp  #1.9:1.10
@@ -76,5 +76,5 @@ SidebarExtension::SidebarExtension( cons
     m_resizeHandle->setCursor(QCursor(Qt::SizeHorCursor));
     connect(p->widget(),SIGNAL(panelHasBeenExpanded(bool)),this,SLOT(needLayoutUpdate(bool)));
-    needLayoutUpdate(true);
+    needLayoutUpdate(false);
     m_resizeHandle->installEventFilter(this);
     m_resizeHandle->setMouseTracking(true);