https://bugs.kde.org/show_bug.cgi?id=521383

--- Comment #3 from cwo <[email protected]> ---
Just adding a new kickoff widget to a panel is enough to trigger it.

I haven't completely tried it out, but I'm pretty certain the issue is that

- Libplasma AppletPopup's updateSize skips the resize if the size was
explicitly set from the config, otherwise it takes the implicitWidth
- Kickoff's implicit width is sidebar width + content width (which is the same
for all views)
- the search view has no sidebar, so a lower width

When we add a kickoff widget, the config doesn't have a size yet (as at that
point it's not determined and written to the config yet).
When we start the search, it removes the sidebar. updateSize does not think it
has a size from the config, so it modifies the popup's size.

After restarting Plasma, it has a size from the config, so it skips the
resizing in the same situation.

The problem didn't happen before 9d55adc0593aebf10a1830c74a8a49c1b454496d,
because then we set a preferred size, which overrules the implicit size. (But
it causes its own issues, such as occasional polish loops apparently?)

If this is correct, it could happen with other widgets too, but they need to
size themselves dynamically the first time like kickoff and not have a
preferred size set, and possibly other conditions (so I'm not sure it applies
to any). 

AppletPopup should probably listen to when the size is saved the first time,
but I'm a bit scared of this breaking other things. Alternatively, we could set
the minimum size in kickoff to handle at least this case (but we might have to
do some manual calculation.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to