Review Request: Auto scroll in kickoff

[Robert Knight]

Hi,

I cannot comment on the idea itself yet but with regards to the code,
there is duplication of code and constants to implement this in UrlItemView and
FlipScrollView.  This should be avoided if possible because it is
likely in future that
some changes might need to be made to the behavior and the implementations
may get out of sync.

Regards,
Robert.

2009/1/17 Andras Keri <andras.ker at freemail.hu>:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.vidsolbach.de/r/333/
> -----------------------------------------------------------
>
> Review request for Plasma.
>
>
> Summary
> -------
>
> This patch allows auto scroll in kickoff which could be handy in the case of long lists. The auto scroll is triggered if the mouse is in the outer half of the top/bottom visible item.
>
> Two things to note:
> - the speed may be not ideal, should be a bit slower,
> - the auto scroll in the case of drag and drop works differently, that could be confusing.
>
> Any comments, suggestions are more than welcome.
>
>
> Diffs
> -----
>
>  trunk/KDE/kdebase/workspace/plasma/applets/kickoff/ui/urlitemview.cpp
>  trunk/KDE/kdebase/workspace/plasma/applets/kickoff/ui/urlitemview.h
>  trunk/KDE/kdebase/workspace/plasma/applets/kickoff/ui/flipscrollview.h
>  trunk/KDE/kdebase/workspace/plasma/applets/kickoff/ui/flipscrollview.cpp
>
> Diff: http://reviewboard.vidsolbach.de/r/333/diff
>
>
> Testing
> -------
>
> On local machine.
>
>
> Thanks,
>
> Andras
>
> _______________________________________________
> Plasma-devel mailing list
> Plasma-devel at kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel
>