[Kde-pim] KDE/kdepim/kontact/plugins

[Rafael Fernández López]

Hi,

> Is this error message actually necessary? Ideally, we would check
> whether dropping is possible while the user is still dragging the
> messages and disable dropping (resulting in the "drop here not
> possible" mouse cursor) if the user drags more than one message.

Yes, that can be checked for sure. I guess that this was initially thought 
this way just in case the first time the user wants to drag&drop does it with 
a thread, and he/she gets a "no-drop" cursor. He/She can think from there that 
it is not possible to drop in there *anything*.

So, yes, I would still vote for the message box error. Apart from that, the 
error teachs you directly that you can't drop threads, and you need to do it 
with single mail items.

> If that's not (easily) possible, then I suggest to write "Drag&drop ..."
> instead of "Drops ...".

Hmm.. probably as Tom said, "Drop" is better, but I think "Drag & drop" is not 
the ideal string. I mean, dragging is never possible, with multiple or single 
elements. The only action you perform on the sidebar are drop actions.

> Also the following message
> +  KMessageBox::sorry( core(), i18n( "Cannot handle drop events of
> type '%1'.", event->format() ) );
> should at most be a debug message (i.e. kDebug() << "Cannot ...")
> because no user will understand this message. Let's try not to confuse
> the user with such technical error messages.

Yes, actually, if canDecodeMimeData() method is consistent with 
processDropEvent() method on the plugin implementation, you should never reach 
this warning (since canDecodeMimeData() should have returned false, and you 
would never have reached processDropEvent()).

So, yes, I agree on moving those warnings from message boxes to kWarnings().


Regards,
Rafael Fernández López.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20080725/04c9b847/attachment.sig>
-------------- next part --------------
_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/