search

aardappel / treesheets

TreeSheets : Free Form Data Organizer (see strlen.com/treesheets)
zlib License
2.5k stars 185 forks source link

Place datetime filter dialog on stack #589

Closed tobiolo closed 5 months ago

tobiolo commented 5 months ago

This code is cleaner.

In addition add also some border around the elements. This looks nicer especially on the border of the dialog.

aardappel commented 5 months ago

very nice! was this a memory leak before? I don't see them being deleted in the old code..

tobiolo commented 5 months ago

very nice! was this a memory leak before? I don't see them being deleted in the old code..

No, at least not detected with Address Sanitizer during runtime. But as it is possible for modal dialogs to create it on the stack, I would rather prefer it anyway.