[Lazarus] Deleting items in OI

[Vojtěch Čihák]

Hi, thanks for reply.
 
Yes, I need something like
1) say to OI thah I'm going to delete TPersistent
2) delete it (my own code)
3) refresh OI (node must disappear)
 
I tried to search other people's code how they do it but I didn't find any.
 
Even worse, there is possibility to delete TPersistent in OI with Delete key which I didn't consider before so I have to redesign several components.
 
V.
 
______________________________________________________________


> Od: Juha Manninen via Lazarus <lazarus at lists.lazarus-ide.org>
> Komu: Lazarus mailing list <lazarus at lists.lazarus-ide.org>
> Datum: 28.11.2017 13:10
> Předmět: Re: [Lazarus] Deleting items in OI
>
On Sat, Nov 25, 2017 at 8:21 PM, Vojtěch Čihák via Lazarus
 <lazarus at lists.lazarus-ide.org> wrote:
 > I just have some more info. Everything was OK to revision 52209.
 > OI was completly broken between 52210 and 52229 (components didn't appear at all).
 > Revision 52230 improved OI but deleting didn't work as described in previous
 > mails, it's up to now.
 
 Yes, those were my commits. Sorry I did not answer earlier, something
 always came up when I planned to study the issue.
 The changes were more or less valid. At least they fixed bugs and
 optimized the updates.
 However your usage of TPropertyEditorHook looks suspicious and
 counter-intuitive:
 
  aHook.PersistentDeleting(aECTab);
  aECTC.DeleteTab(aECTC.TabIndex);
  aHook.SelectOnlyThis(aECTC.Tabs);  { force the OI to refresh }
 
 Calling aHook.DeletePersistent() would be intuitive. It that is not
 possible in your case, then we should improve the API.
 Something like "StartDeletingPersistent()" + "EndDeletingPersistent()"
 pair maybe.
 How about calling aHook.Modified?
 What would be the most intuitive API in your opinion?
 
 Juha
 -- 
 _______________________________________________
 Lazarus mailing list
 Lazarus at lists.lazarus-ide.org
 https://lists.lazarus-ide.org/listinfo/lazarus <https://lists.lazarus-ide.org/listinfo/lazarus>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.lazarus-ide.org/pipermail/lazarus/attachments/20171129/8d0c7b97/attachment-0001.html>