No subject
Rafael Fernández López
ereslibre at kde.org
Tue Aug 4 14:15:11 BST 2009
Ok, I think you mean the attached patch.
Another thing I did when rewriting this was to start it right from the very
beginning.
I don't think this kind of feature should be provided by the view, and is
more a task of the delegate, for this reason, the attached patch is wrong
from my point of view, and what we should do is to:
a) Modify KFileItemDelegate to take into account the possibility of being
disabled.
or
b) Create another delegate that inherits KFileItemDelegate and does the
right thing in this very case.
I will see later (on the night) if I can write a patch for KFileItemDelegate
and if it makes sense.
Regards,
Rafael Fernández López.
PS: Please, please, please always write subjects for mails.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20090804/259d6388/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kdelibs.diff
Type: application/octet-stream
Size: 1380 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20090804/259d6388/attachment.obj>
More information about the kde-core-devel
mailing list