Bug 119560 - Window Thumbnails in pager applet are disabled after re-login
Summary: Window Thumbnails in pager applet are disabled after re-login
Status: RESOLVED FIXED
Alias: None
Product: kicker
Classification: Plasma
Component: minipagerapplet (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Aaron J. Seigo
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-01-05 11:46 UTC by clcevboxvjeo
Modified: 2006-07-23 19:20 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
possible fix? (1.13 KB, patch)
2006-07-23 01:25 UTC, Aaron J. Seigo
Details

Note You need to log in before you can comment on or make changes to this bug.
Description clcevboxvjeo 2006-01-05 11:46:30 UTC
Version:            (using KDE KDE 3.5.0)

When you enable showing window thumbnails in kicker's pager appletm then log out and log in, the setting is lost. Thumbnails are turned off.
Comment 1 clcevboxvjeo 2006-03-07 16:15:47 UTC
The bug is also in KDE 3.5.1
Comment 2 Vinay S Shastry 2006-05-18 11:33:02 UTC
Also present in 3.5.2

(OS: Linux, Distribution: Archlinux, Package: Official distribution packages)
Comment 3 Vinay S Shastry 2006-07-22 23:18:33 UTC
This looks like 800x600 and less resolution specific ? I just went through the code... and heres what it says..
*line 101*

    if (desk.width() <= 800)
    {
        m_settings->setPreview(false);
    }


I changed my monitor res to 1024x786 (very horrible refresh rates here 15"monitor..pls dont laugh..i'm stuck with it. heh) and previews were on. Previews look great on 800x600.. I wonder what made the devs add that forced check. Is it possible for Aaron to change that 800 to.. say 640 ? 8)

I'm sure that changing it to something less than 800 will do the job.. but gonna recompile and make sure.
Comment 4 Vinay S Shastry 2006-07-22 23:24:35 UTC
Forgot to mention the file name. Its kdebase-3.5.3/kicker/applets/minipager/pagerapplet.cpp
Comment 5 Vinay S Shastry 2006-07-23 00:01:55 UTC
Yep, thats it. I recompiled with that modification and it works. Hope the forced preview disabling will be removed/modified.
Comment 6 Aaron J. Seigo 2006-07-23 01:25:48 UTC
Created attachment 17093 [details]
possible fix?

could you please try the attached (untested) patch? i can't test it right now
against 3.5 and would like to have this fix in for 3.5.x. it's a bit of a hack,
but it should set the default to false, preserving the default as false but
allowing the user to override it. thanks =)
Comment 7 Aaron J. Seigo 2006-07-23 19:20:57 UTC
SVN commit 565544 by aseigo:

allow manually turning on of previews in the pager at low screen
resolutions
thanks to vinay shastry for testing the patch.
BUG:119560


 M  +10 -7     pagerapplet.cpp  


--- branches/KDE/3.5/kdebase/kicker/applets/minipager/pagerapplet.cpp #565543:565544
@@ -87,6 +87,16 @@
       m_contextMenu(0),
       m_settings( new PagerSettings(sharedConfig()) )
 {
+    int scnum = QApplication::desktop()->screenNumber(this);
+    QRect desk = QApplication::desktop()->screenGeometry(scnum);
+    if (desk.width() <= 800)
+    {
+        KConfigSkeleton::ItemInt* item = dynamic_cast<KConfigSkeleton::ItemBool*>(m_settings->findItem("Preview"));
+        if (item)
+        {
+            item->setDefaultValue(false);
+        }
+    }
     m_settings->readConfig();
     m_windows.setAutoDelete(true);
     TaskManager::the()->trackGeometry();
@@ -96,13 +106,6 @@
 
     setFont( KGlobalSettings::taskbarFont() );
 
-    int scnum = QApplication::desktop()->screenNumber(this);
-    QRect desk = QApplication::desktop()->screenGeometry(scnum);
-    if (desk.width() <= 800)
-    {
-        m_settings->setPreview(false);
-    }
-
     m_kwin = new KWinModule(this);
     m_activeWindow = m_kwin->activeWindow();
     m_curDesk = m_kwin->currentDesktop();