Thanks for all the feedback, guys!
hayasidist wrote: ↑Thu Jan 04, 2024 2:52 pm
I'll give it a whirl (and have a deeper think about having "the same script" as a menu and/or tool button... )
Good! It never hurts... Just in case there was some aspect I have not taken into account or whatever.
synthsin75 wrote: ↑Fri Jan 05, 2024 12:49 am
Building on the idea of showing/hiding tools, seems you'd be able to make a single tool (so it's always available in the tool window) that could store tool "profiles"... groups of tools...
One of my first thoughts was indeed create a tool or window to provide such kind of management, but when the current approach raised it was simply too tentative for its simplicity. I was amazed with the fact only one line were enough to can make it work that way

, and with the functionality included in the own tool there is no dependencies, but for more advanced possibilities like those you name it well could worth something more elaborated, of course.
SimplSam wrote: ↑Fri Jan 05, 2024 1:42 am
HaHa. I didn't eve realise that that was a Preview. I was thinking: what the hell is that 'stupid' button for? In an ideal
world that would be bigger - or with an option to place it where the Palette is placed (
maybe if you click on the Preview thumbnail toggle).

Sincerely, I didn't expect it to get so much attention, overall after I had to disable fills in the last minute due to a weird viewport behavior that I'll have to investigate with calm... The problem is I didn't get to copy/paste or transfer stuff from
M_Mesh's objects to
LM_MeshPreview ones (because that would simply have been too easy?

) and I had to resort to a workaround involving
CombineShapes() function that seemed to work, but turned out for some reason interferes with existing Liquid Shapes, if any, after undo... But besides that, what I'm not sure is if I follow you with this part: "an option to place it where the Palette is placed", Sam, do you mean bellow where the Raise/Lower buttons are would be a better place for it? Becase at some point I considered that, but as I went adding buttons there the current placement started to seem a better place to me, since the widget wouldn't be aligned with the selected shape anyway (which of course would be the ideal).
SimplSam wrote: ↑Fri Jan 05, 2024 1:42 am
Funky Palettes (
even though I think the slider will prove to be redundant)
Really? But if the SLIDER was indeed the more interesting part to me! I mean, it's the key to make them really functional (to the extent possible) and not only a static palette as we currently have. Maybe the screen recording I included gives a wrong idea of it, but the typical use case is not intended to go changing between Swatches as shown there, but can use the one you prefer (or the one more appropriate for the case) and can get the
whole range of colors plus saturation by means of the slider, without witch you'd have to have different Swatches for each primary color, and even though the color range would be much more limited. But of course I understand that, since the color precision is not warrantied due to the obvious limitations, one can not find it as useful as it could be...
SimplSam wrote: ↑Fri Jan 05, 2024 1:42 am
One thing that doesn't seem to be working is the Large Icons. Icons are just the same size.
But the size of the buttons should be larger even though the icons remains the same (I added such clarification afterwards to the notes), that's what you were referring, isn't it? Or they don't change for you at all? Since the change requieres the window to be reopening, I could have included a "@2x" set of images and that would work for sure, but for the moment I thought it could be enough to simply provide a bigger clickable area with, mainly, pencil users in mind. Oh, and larger buttons should provide a little bigger shape preview, BTW.
SimplSam wrote: ↑Fri Jan 05, 2024 1:42 am
Also ... for me - the 'Style' cursors are 'unnecessary' - bordering on annoying. These (and Swatch) Drop Down Menus might be better as ImagePopupMenu's.
And I kind of agree

, but the reason behind is I wanted all them in only one row and the width is so limited that I had to discard the use of ImagePopupMenu (since with the icon itself plus the arrow already takes too much space, and I wanted at least some letters of the selected items to be shown). So I thought the cursors could help on clarifying what you are dealing with instantly, maybe with more unexperienced users in mind for which dealing with Styles are already difficult enough... But yeah, nothing that totally convinced to me either, so I well may change that, or provide a way to disable that kind of superfluous (for more advanced users at least) stuff.
SimplSam wrote: ↑Fri Jan 05, 2024 1:42 am
p.s. You may also need to set/reset some sort of 'update lock' around the use of LM_ImageTextList:RemoveItem() in a loop - as it calls HandleMessage each time. This can account for slow updates when switch Layers for example.
Indeed, I've tried to be very cautious regarding that and very specially where loops are involved, but of course I may have oversight things... I'll review everything having to do with that and see, but if you've detected something in particular it could be of great help to know where, if possible, of course.
SimplSam wrote: ↑Fri Jan 05, 2024 1:42 am
There appears to be a problem when deleting selected Point KeyFrames. When the tool is open and points selected - it always deletes the Point/s - no matter where the window focus is.
Yeah... I also detected that at some late point, but I kind of assumed it could be due to some kind of problem with the dialogs system itself and didn't investigated it much far... I'm not very confident about it, but I wish I was wrong and it's something I can fix, though!
Sorry for the long post, I didn't realice till I pressed the Preview button... and thank you, very much indeed, again for all the provided feedback and added thoughts!