Re: [Abook-devel] patchset for review

[Thorsten W. <ed...@th...>]

On Tue, May 08, 2012 at 10:50:41PM +0200, Raphaël Droz wrote:
> before pushing to the main abook repository I pushed 9 reworked
> commits here [1]
> If nothing is wrong with them I hope they can make their way in
> the main repository at sf.net.
> 
> [1] https://gitorious.org/drzraf/abook/commits/master

The only strange thing is scrolling, because it just moves the selected
line-marker instead of really scrolling the list (like it is known from
vim, ncmpcpp or real gui-applications). You'll find a patch
0001-Scroll-whole-list-on-mouse-wheel-action.patch attached that let's
the actual whole list scroll.

I didn't push it to my repository because the names of the new functions
and maybe even its behaviour is worth discussing. What's your opinion
about these two points?

Thorsten