Bug 180440 - Plasma pager is a bit too big
Summary: Plasma pager is a bit too big
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Unmaintained
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-01-12 11:26 UTC by Jan Mette
Modified: 2009-02-17 22:17 UTC (History)
5 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
pager screenshot (14.71 KB, image/png)
2009-01-12 11:27 UTC, Jan Mette
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jan Mette 2009-01-12 11:26:10 UTC
Version:            (using Devel)
Compiler:          GCC 4.3.2 
OS:                Linux
Installed from:    Compiled sources

Hi!

i just updated my 4.2rc1 build (from the 4.1.96 tag) with a newer one from the KDE 4.2 branch -> rev. 909644

With the older build, the pager was sized correctly in the panel. With the newer build, it seems to be cut off at the bottom.

This doesnt change when i resize the panel or change settings for the pager. Also deleting the plasma configs didnt work out.

Check the screenshot i'll attach now.

Oh, and thanks for all the hard work on KDE, i just love it :)

Greetings

Jan
Comment 1 Jan Mette 2009-01-12 11:27:36 UTC
Created attachment 30179 [details]
pager screenshot

its cut-off at the bottom
Comment 2 Anne-Marie Mahfouf 2009-01-12 11:49:19 UTC
Also in trunk. Furthermore default is 1 row while it always was 2 rows.
Comment 3 Marco Martin 2009-01-12 21:17:12 UTC
it's after revision 908477
it reomoves all the margins from the size calculation, code is simpler but doesn't seem to work very well...
Comment 4 Aaron J. Seigo 2009-01-12 21:54:40 UTC
@Anne-Marie:
> Furthermore default is 1 row while it always was 2 rows.

really? it's 2 rows here. are you seeing this in the config dialog, or in the actual painting of the pager, or? because i'm seeing 2 rows in both situations...

@Marco: the margins are actually already added to the m_rects calculations in Pager::recalculateGeometry. the margins that that r#908477 removed were resulting in adding the margins *twice*, but only during mouse interactions, and therefore all the mouse interaction was off.

anyways.. this bug only shows itself with pager rows == 1 (here anyway).. i'll look into a fix for that.
Comment 5 Aaron J. Seigo 2009-01-12 21:59:20 UTC
SVN commit 910229 by aseigo:

add the margins to the width/height; we do those margins, only in the mouse interaction we don't want them (re-)added
BUG:180440


 M  +2 -2      pager.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=910229
Comment 6 Aaron J. Seigo 2009-01-12 22:00:46 UTC
SVN commit 910230 by aseigo:

add the margins to the width/height; we do those margins, only in the mouse interaction we don't want them (re-)added
CCBUG:180440


 M  +2 -2      pager.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=910230
Comment 7 Jan Mette 2009-01-12 23:52:11 UTC
Just rebuilt kdebase-workspace (rev 910273).

It looks fine now, thank you :)
Comment 8 Anne-Marie Mahfouf 2009-01-13 08:18:52 UTC
Fixed for me as well. I had a new install and $KDEHOME and the pager as default was all 4 desktops on 1 row.
This morning with Revision: 910338 it's all back to normal, default is on 2 rows and position is perfect!
Comment 9 Olaf Lenz 2009-02-03 13:31:11 UTC
Unfortunately, the fix seems to have broken the resizing behaviour of the pager when it is used on the desktop (see bug 182487)...
Any ideas how both bugs can be fixed at once?
Comment 10 Robin Pedersen 2009-02-03 14:24:07 UTC
Reopened. See bug 182487.
Comment 11 Robin Pedersen 2009-02-05 18:02:44 UTC
SVN commit 921782 by rpedersen:

Handle margins correctly both when pager is on desktop and in panel

This should result in correct behavior in both cases. But IMHO, this
requires a better solution..

BUG: 180440
BUG: 182487

 M  +8 -2      pager.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=921782
Comment 12 Robin Pedersen 2009-02-05 18:05:16 UTC
SVN commit 921784 by rpedersen:

backport 921782

CCBUG: 180440
CCBUG: 182487


 M  +8 -2      pager.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=921784
Comment 13 Robin Pedersen 2009-02-17 22:17:08 UTC
SVN commit 927574 by rpedersen:

Handle margins correctly both when pager is on desktop and in panel

Using Plasma::Desktop did not work, now using Plasma::Floating instead
which seems to work for me.

CCBUG: 180440
CCBUG: 182487

 M  +2 -2      pager.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=927574