Bug 226429 - Plasma ScrollWidget issues when resizing [was: The device notifier has layouting problems]
Summary: Plasma ScrollWidget issues when resizing [was: The device notifier has layou...
Status: RESOLVED WORKSFORME
Alias: None
Product: plasma4
Classification: Unmaintained
Component: general (show other bugs)
Version: unspecified
Platform: Ubuntu Unspecified
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
: 226310 (view as bug list)
Depends on:
Blocks:
 
Reported: 2010-02-12 00:55 UTC by Albert Astals Cid
Modified: 2011-08-01 20:11 UTC (History)
3 users (show)

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


Attachments
Screnshot showing the problem (36.83 KB, image/png)
2010-02-12 00:56 UTC, Albert Astals Cid
Details
Original state (30.83 KB, image/png)
2010-06-20 14:29 UTC, Albert Astals Cid
Details
After expanding the last device (47.10 KB, image/png)
2010-06-20 14:30 UTC, Albert Astals Cid
Details
After unexpanding the last device (24.20 KB, image/png)
2010-06-20 14:30 UTC, Albert Astals Cid
Details
Screencast showing how to reproduce the issue (533.71 KB, video/ogg)
2010-06-20 20:04 UTC, Jacopo De Simoi
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Albert Astals Cid 2010-02-12 00:55:40 UTC
Version:            (using KDE 4.4.0)
Installed from:    Ubuntu Packages

This is how i can reproduce it:
 * Have 3 items in the device notifier
 * Click on the last one
 * It expands asking if i want to open it with gwenview or dolphin
 * Also creates a scrollbar
 * Now close it to unexpand the item
 * The scrollbar is gone
 * But items are not correctly repositioned and the top item is painted over the "Devices connected recently"
Comment 1 Albert Astals Cid 2010-02-12 00:56:55 UTC
Created attachment 40705 [details]
Screnshot showing the problem
Comment 2 Jacopo De Simoi 2010-06-20 12:41:03 UTC
*** Bug 226310 has been marked as a duplicate of this bug. ***
Comment 3 Albert Astals Cid 2010-06-20 14:26:23 UTC
Updated info from me, now instead of the device being overlapped with the text the device is "lost" in the begininng of the list, but no scrollbar is pressent, but you can still scroll with the mouse wheel to recover the device icon.

Doing a series of screenshots so you can see what i mean.
Comment 4 Albert Astals Cid 2010-06-20 14:29:43 UTC
Created attachment 48162 [details]
Original state
Comment 5 Albert Astals Cid 2010-06-20 14:30:11 UTC
Created attachment 48163 [details]
After expanding the last device
Comment 6 Albert Astals Cid 2010-06-20 14:30:50 UTC
Created attachment 48164 [details]
After unexpanding the last device
Comment 7 Jacopo De Simoi 2010-06-20 18:36:37 UTC
I could finally reproduce the issue; i need an extra step with respect to the ones given by Albert:

 * Have 3 items in the device notifier
 * Click on the last one
 * It expands asking if i want to open it with gwenview or dolphin
 * Also creates a scrollbar
 * Now use the scrollwheel to scroll up and down; 
 * While the widget is still moving click on the itemBackground to close and unexpand the item
 * The scrollbar is gone
 * But items are not correctly repositioned and the top item is painted over
the "Devices connected recently"

@notmart: 
 1) Can you see why something bad could happen if the widget changes size during an animation?
 2) Can you reproduce with 4.7?
Comment 8 Jacopo De Simoi 2010-06-20 20:04:49 UTC
Created attachment 48169 [details]
Screencast showing how to reproduce the issue
Comment 9 Marco Martin 2010-06-21 14:11:59 UTC
SVN commit 1140731 by mart:

stop animations when the inner widget spontaneously resize
it should fix 226429 and a very similar issue in the netbook, please test :)
CCBUG:226429


 M  +1 -0      scrollwidget.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1140731
Comment 10 Marco Martin 2010-06-21 16:47:47 UTC
SVN commit 1140801 by mart:

ensure the main widget is visible after its resize
CCBUG:226429


 M  +1 -0      scrollwidget.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1140801
Comment 11 Marco Martin 2010-06-23 20:51:19 UTC
after those 2 patches the problem doesn't seem to be reproducible by both of us.
if it persists with a chackout more recent than those, please reopen
Comment 12 Jacopo De Simoi 2010-06-25 15:43:19 UTC
Argh, I just managed to reproduce using latest trunk; the way to reproduce it is indeed more complicated and might be related to another scrollwidget bug.
To reproduce, assuming the notifier is on the panel:

1) Open the device notifier with a certain number of devices (3 should be enough)
2) Click on the last device to expand it; make sure that the vertical scrollbar appears.
3) Resize the device notifier (alt-right drag) to make it big enough so that the scrollbar disappears
4) By making it just a little bit smaller, I can get to a state where scrolling is enabled (i.e. you see the shadows at the top and bottom) but there is no scrollbar (???)

5) Now acting as in my previous screencast (i.e. scrolling + click to unexpand) is sufficient to trigger this bug 100% of the times.

---

Step 4) is the fishy step, this should just not happen I guess…
Comment 13 Aaron J. Seigo 2011-07-30 17:36:49 UTC
is this still open and a real issue? i'd like to see this closed as one of wontfix or fixed. it shouldn't remain open.
Comment 14 Albert Astals Cid 2011-08-01 20:11:18 UTC
As original reported, i'm happy enough