[PATCH] Fixing KWidgetItemDelegate::focusedIndex()
Aurélien Gâteau
aurelien.gateau at free.fr
Fri Oct 17 07:30:30 BST 2008
Rafael Fernández López wrote:
> Hi,
>
>> Attached patches use an alternative approach: instead of tracking
>> the focused index, I just let KWidgetItemDelegate::focusedIndex()
>> figure out which one is focused when it's called. It's much simpler and
>> seems to work well. Is it ok for you?
>
> Yes, it seems fine. I wonder why I let the switch() survive on the
> "this-switch-is-no-longer-useful" patch...
Not your mistake: The only other case: of the switch() was removed by my
first patch. I decided to do the clean in a separate patch to make the
first one easier to review.
> Please commit.
OK. Will do.
Aurélien
More information about the kde-core-devel
mailing list