Fwd: KDirOperator patchlets

Peter Penz peter.penz at gmx.at
Sun Jul 20 18:56:28 BST 2008


On Sunday, 20. July 2008 16:15:32 Daniel Molkentin wrote:
[...]
> Firtst thing I noticed (after getting it to compile at all) was that
> KDirOperator doesn't send out the fileHighlighted() signal e.g. when
> changing the current item. It only does that in multi-selection mode or
> when no selection is set at all. See attached
> kdiroperator-fileHighlighted.patch.
>
> The second thing is that pressing return on the current item does not emit
> fileSelected() but fileHighlighted(). I'm wondering if this was an accident
> or done on purpose. See attached kdiroperator-fileSelected.patch.
>
> I already committed the former patch to HEAD. Ideally, I'd like to see both
> in 4.1.

I have not tried the patch yet, but it looks good. Maybe Rafael (added to CC) 
could give a confirmation too...

Cheers,
Peter

> Thanks for any comments,
> Carsten
>
> --
> Carsten Pfeiffer
>
> -------------------------------------------------------






More information about the kde-core-devel mailing list