Review Request: Save scrollbar position on plasma exit

Marco Martin notmart at gmail.com
Wed Jun 20 09:50:25 UTC 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104258/#review14906
-----------------------------------------------------------


what is the status of this? has been pushed? is pretty old, should be either dropped or submitted but not left dangling

- Marco Martin


On March 16, 2012, 4:31 p.m., Ignat Semenov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104258/
> -----------------------------------------------------------
> 
> (Updated March 16, 2012, 4:31 p.m.)
> 
> 
> Review request for Plasma, Aaron J. Seigo, Marco Martin, and Fredrik Höglund.
> 
> 
> Description
> -------
> 
> This patch implements scrolbar position saving on plasma exit. The change is fairly trivial, however, due to the fact that the view is not populated and layouted immediately simply scrolling to the desired position on creating the view does not work. Instead a signal is emitted on finishing the item layout, when the view has a valid size and the scrollbar has a valid range. The signal is connected to a slot which scrolls the view to the desired position and then disconnects the signal. For the user, a public function in AbstractItemView is introduced, which performs the connection.
> 
> The only problem is that ListView turned out not to have any layout method. It just paints the items one by one, calculating their position on the fly, so I put the signal at the end of updateScrollbar to ensure the scrollbar range is valid. Maybe it should go into the "if (max>0)" branch?
> 
> 
> This addresses bug 261139.
>     http://bugs.kde.org/show_bug.cgi?id=261139
> 
> 
> Diffs
> -----
> 
>   plasma/applets/folderview/abstractitemview.h aa68b90 
>   plasma/applets/folderview/abstractitemview.cpp 3debb70 
>   plasma/applets/folderview/folderview.h 4e441eb 
>   plasma/applets/folderview/folderview.cpp a94ce87 
>   plasma/applets/folderview/iconview.h 12e93b3 
>   plasma/applets/folderview/iconview.cpp 5c4e086 
>   plasma/applets/folderview/listview.cpp 94efe44 
> 
> Diff: http://git.reviewboard.kde.org/r/104258/diff/
> 
> 
> Testing
> -------
> 
> Tested both the icon view and the list view, works fine.
> 
> 
> Thanks,
> 
> Ignat Semenov
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20120620/d02bacdf/attachment.html>


More information about the Plasma-devel mailing list