Fwd: KDirOperator patchlets

[Carsten Pfeiffer]

Am Samstag, 26. Juli 2008 schrieb Peter Penz:

Hi Peter,

> Ups, the patch has a sideeffect which I think we all missed: It is not
> possible anymore selecting more than one file in e. g. Kate or KWrite.
>
> Reverting commit 836195 from the 4.1 branch brings back the original
> functionality.  You write in your mail that included the 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."
>
> It seems like this has been done on purpose to make it possible selecting
> more than one file. May I revert this patch on KDE 4.1 and trunk? Well it
> will be too late for 4.1.0 I assume, but it's for sure no showstopper if
> the fix will be in 4.1.1...

I assume that there are not many clients of KDirOperator besides KFileDialog 
in KDE 4, so a working KFileDialog is obviously most important.

However simply reverting the patch is not a solution either. I see two 
possibilities:

- emit fileSelected(const KFileItemList&) with the current selection, or
- let the client, e.g. KFileDialog honor the selection instead of only the 
argument in multiselection mode (that's how it worked in 3.5, I think)

I can provide a patch if you want. Just revert the patch for 4.1.0, if it's 
not too late.

Cheers,
Carsten
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 447 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080728/23591daf/attachment.sig>