Bug 197055 - battery monitor widget forgets it's size
Summary: battery monitor widget forgets it's size
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Plasma
Component: widget-battery (show other bugs)
Version: unspecified
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
: 200719 (view as bug list)
Depends on:
Blocks:
 
Reported: 2009-06-18 20:23 UTC by Bob K
Modified: 2009-12-22 21:52 UTC (History)
7 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Bob K 2009-06-18 20:23:15 UTC
Version:            (using KDE 4.2.90)
OS:                Linux
Installed from:    Ubuntu Packages

How to reproduce:
- Put "Battery monitor" widget on the desktop
- Resize it
- Log out
- Log in
Comment 1 Artur Souza (MoRpHeUz) 2009-06-18 20:32:24 UTC
I couldn't reproduce with trunk (kde 4.3).
Comment 2 Dario Andres 2009-06-21 01:22:22 UTC
Does any other widget added during the same session show the same behaviour ?
Thanks
Comment 3 Bob K 2009-06-21 09:41:39 UTC
(In reply to comment #2)
> Does any other widget added during the same session show the same behaviour ?
> Thanks

No. I have tried out now that i removed some of my widgets and re-added them again plus i added some other widgets and last i resized both. Logout, login: only the battery monitor forgotted it's size.

When Artur Souza said that he can't reproduce the bug, i tried to find some commit log about this, but i did'nt found any...
Comment 4 Bob K 2009-07-03 15:48:55 UTC
Now on 
Qt: 4.5.0
KDE: 4.2.95 (KDE 4.2.95 (KDE 4.3 RC1))
kde4-config: 1.0

And the problem still exists!
Comment 5 Dario Andres 2009-07-15 19:16:35 UTC
If you restart Plasma without finishing the user session (on Konsole, run "kquitapp plasma-desktop && sleep 3 && plasma-desktop"), does this also happen ?
Comment 6 Dario Andres 2009-07-15 19:38:56 UTC
Just tested here restarting plasma:

Qt: 4.5.2 (KDE-Qt git commit 46a247a2c9a8c0c4456a02f6a0922d859d88fe76
        Date:   Fri Jun 26 13:45:37 2009 +0200)
KDE: 4.3.60 (KDE 4.3.60 (KDE 4.4 >= 20090706))
kdelibs svn rev. 996016 / kdebase svn rev. 996016
on ArchLinux i686 - Kernel 2.6.30

(I removed my plasma* config files from ~/.kde4/share/config)
The Battery widget placed on the Desktop do not remember its size (it remembers the position)
The geometry of the applet is properly save to the rc file after resizing it and locking the widgets. But after a plasma restart it was 128x128 again.

Battery::Battery calls "resize(128, 128);"
Comment 7 Beat Wolf 2009-07-15 19:55:31 UTC
Are you going to fix it?
Comment 8 Dario Andres 2009-07-15 19:57:57 UTC
My last comment is not completely true... After realoding plasma and when the
battery is 128x128 again, the rc file still contains the old size value. So,
the size is being saved but not properly restored.

@Fix it: I think it is out of my scope, I lack Plasma knowledge to do this.
Comment 9 Dario Andres 2009-07-19 22:45:50 UTC
*** Bug 200719 has been marked as a duplicate of this bug. ***
Comment 10 Bob K 2009-07-23 16:19:06 UTC
Now i don't know, that i need to do anything?
Btw, now on:

Qt: 3.3.8b
KDE: 3.5.10
kde-config: 1.0

And the problem is still exists
Comment 11 Bob K 2009-07-23 16:21:17 UTC
Sorry, i've accidentally copied kde-config's output instead of kde4-config's
output, so again:

Qt: 4.5.0
KDE: 4.2.96 (KDE 4.2.96 (KDE 4.3 RC2))
kde4-config: 1.0
Comment 12 Sergei Danilov 2009-08-06 13:06:09 UTC
I can confirm the same behaviour for the kde 4.3 battery applet.Steps are:
create new user testuser and login under kde 4.3
drag battery applet to the desktop.Byt default it's huge.
Make it smaller and turn option to display the battery charge percentage
Logout
Login again
batter widget is again hude and w/o percentage shown
os gentoo linux gentoo-sources-2.6.30-r4
x86_64-pc-linux-gnu-gcc
qt 4.5.2
Comment 13 Bob K 2009-08-06 14:07:24 UTC
(In reply to comment #12)
[...]
Yes, i have upgraded to 4.3 and the problem still occurs. With the folder view too.
Comment 14 Sergei Danilov 2009-08-06 14:26:47 UTC
for me folder view is Ok.Only batter monitor.
maybe it's because of I'am using new user for this experiment
Comment 15 Magnus Kessler 2009-08-19 19:54:32 UTC
I can easily reproduce this with version 4.3.0, by putting the battery applet between the panel and another plasma applet. I traced this down to revision 976141 (http://websvn.kde.org/?view=rev&revision=976141). Presumably placing the applet close to other plasma managed objects triggers the constraint code, which in turn makes it appear in "enormous" size. Reverting the code change from revision 976141 lets the applet keep its size after logout and login.
Comment 16 Dario Andres 2009-12-10 00:22:22 UTC
@Sebas: can you review comment 15 and this bug ? Thanks
Comment 17 Sebastian Kügler 2009-12-22 21:52:38 UTC
I've reverted this patch in r1065250.