Bug 286923 - kget plasmoids don't have resize handles
Summary: kget plasmoids don't have resize handles
Status: RESOLVED FIXED
Alias: None
Product: kget
Classification: Applications
Component: plasma widget (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: KGet authors
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-18 14:04 UTC by Todd
Modified: 2013-06-29 16:31 UTC (History)
1 user (show)

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

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Todd 2011-11-18 14:04:32 UTC 
Version:           unspecified (using KDE 4.7.3) 
OS:                Linux

On hover, plasma applets normally show a bar on the left or right that let you move, resize, rotate, configure, or remove the widget.  The two kget plasma applets, pie chart and bar chart, do not have these bars, making it impossible to resize the widget.

Reproducible: Always

Steps to Reproduce:
1. Put a kget bar chart or pie chart plasma applet on a desktop containment
2. Hover your mouse over the applet

Actual Results:  
No bar appears

Expected Results:  
A bar appears on the side of the widget closer to your mouse
Comment 1 Matthias Fuchs 2011-12-28 14:55:41 UTC 
I can confirm that problem.

To me it looks like
if (constraints & Plasma::SizeConstraint) {
in kgetapplet causes the problem.
What do you think Lukas?

Maybe (@Lukas) we should try to fix this issue for 4.8 and then abort these plasmoids as they are now and port them to Qt Quick?
Comment 2 Lukas Appelhans 2012-01-03 13:54:36 UTC 
Yes, I agree with the porting to QtQuick.

About this issue:
Not sure if that check is the problem, as it should only get triggered when the applet gets resized, it's not an indicator that it can get resized. (And I just tested this by commenting the lines out, does not change anything.)

I looked into it once already, but didn't find the cause of it. (I was focussing on the event handling though...)

Lukas
Comment 3 Lukas Appelhans 2013-06-29 16:31:19 UTC 
I just fixed this in commit 8e216b2eb4f730393ddde0166a4d49592e003f0e

Lukas