Bug 336070

Summary: Kickoff keeps growing
Product: [Plasma] plasmashell Reporter: Aleix Pol <aleixpol>
Component: Application Launcher (Kickoff)Assignee: Sebastian K├╝gler <sebas>
Status: RESOLVED FIXED    
Severity: normal CC: bhush94, notmart
Priority: NOR    
Version: master   
Target Milestone: 1.0   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed In:

Description Aleix Pol 2014-06-11 10:11:04 UTC
I have this impression that every time I restart the menu is slightly bigger. That's the current values in the config:
[Containments][1][Applets][2][Configuration][PopupApplet]
DialogHeight=550
DialogWidth=650

This is almost 1/2 my screen, it used to be ~1/4.

Reproducible: Always

Steps to Reproduce:
1. Start plasma shell
2. Shut down
3. Go to 1
Comment 1 Bhushan Shah 2014-06-11 12:11:04 UTC
I once reported this to IRC, once that happened to me but after that never..
Comment 2 Marco Martin 2014-06-11 16:01:53 UTC
does the DialogHeight/DialogWidth values stored in the config file grow as well?
Comment 3 Marco Martin 2014-06-11 16:13:36 UTC
can't reproduce it atm. if i remove the popupapplet section of kickoff it sets it as 555 of height and never moves frmo here between restarts.

can you reproduce the growth if you remove or alter its saved size?
otherwise looks like simply minimum size of the applet that changed over time
Comment 4 Aleix Pol 2014-06-11 21:32:53 UTC
Now the width is 658.
Comment 5 Vishesh Handa 2014-06-20 15:24:57 UTC
Git commit d3de99eb1e59c30a7e0fa53c81dcf0af5cb35d67 by Vishesh Handa.
Committed on 20/06/2014 at 14:36.
Pushed by vhanda into branch 'master'.

AppletQuickItem: Do not remember the popup dialog size

The popup dialog can currently never be resized, so it doesn't make
sense to record its height in a config file. Additionally, this also
causes problems when applet writers change the size of their plasmoid.
Since the old size is saved in the config file, it is shown with the old
dimensions instead of the new ones.

REVIEW: 118849

M  +0    -39   src/plasmaquick/appletquickitem.cpp
M  +0    -1    src/plasmaquick/private/appletquickitem_p.h

http://commits.kde.org/plasma-framework/d3de99eb1e59c30a7e0fa53c81dcf0af5cb35d67