[Chandler-dev] bug #6868

John Anderson john at osafoundation.org
Wed Oct 4 18:24:19 PDT 2006


Hi Jeffrey:

I spent a few minutes looking into #6868

It looks deleting the selection in the table broke when alec added 
onRemoveEvent and onDeleteEvent, which handled the delete instead of 
table's DeleteSelection.

You're right that we need to send SelectItems in onRemoveEvent and 
onDeleteEvent, like we do in DeleteSelection. We probably don't need the 
selectedCollection in the SelectItems, though -- at least not for the 
detail view. I have a bug to remove it everywhere from the SelectItems 
event.

We should also remove DeleteSelection from table and make sure that we 
didn't miss anything else in our implementation that replaced it in 
PimBlocks.

John


More information about the chandler-dev mailing list