Forum > LCL

Interesting discovery with TListView with OwnerData checked and Adding items.

<< < (3/3)


--- Quote from: wp on March 18, 2024, 05:18:55 pm ---
--- Quote from: dsiders on March 18, 2024, 04:47:01 pm ---
--- Quote from: wp on March 18, 2024, 11:09:16 am ---I don't like the Beep. Is a Beep used within the IDE anywhere else?
--- End quote ---

Not in the IDE specifically, but in LCL bring-to-front flash-and-beep behavior.

--- Quote from: wp on March 18, 2024, 11:09:16 am ---And I'd also replace the Application.MessageBox by MessageDlg even it if makes the unit depend on the Dialogs unit. But Dialogs is used already by PropEdits. So, this should not make a difference. Or am I missing something?
--- End quote ---

No, you're not missing anything. I just took the path of least resistance.

--- End quote ---
OK when I commit it?

--- End quote ---

Here's the modified version.

Committed (, will be backported to Laz/Fixes and the next bug-fix release v3.4

I would like to propose another solution to this problem. I suggest printing a message when trying to set OwnerData if the items are not empty. The user should be warned about the loss of items and delete them themselves. This is a more correct way.

Maybe I'll make a patch myself, but I don't have time yet. Feel free to do it yourself.


[0] Message Index

[*] Previous page

Go to full version