porres / pd-else

ELSE - EL Locus Solus' Externals for Pure Data
Do What The F*ck You Want To Public License
276 stars 36 forks source link

Toggle objects attached to "Presets" object affected when using the "save" preset method. #2102

Open pereshaped opened 4 weeks ago

pereshaped commented 4 weeks ago

When trying to save a preset (using a save message box) in which there are toggle object involved, the save button flips the toggles. I noticed also that they are affected when selecting a preset using either a radio button or a number box.

You can see the behaviour here: https://www.facebook.com/502332142/videos/988104119433511/

porres commented 4 weeks ago

Hi, like I said, I need a minimal patch (meaning a very simple patch with the minimum elements that show the problem), uploaded here as a zipped file, so I can see if I can reproduce and then debug it...a

pereshaped commented 4 weeks ago

Here is the patch. Now!. On first creating the patch and testing it exhibited the described behaviour. Now that I have loaded the patch again it seems ok and not exhibiting the described problem.

Macbook Air 2011 OsX High Sierra PD 0.54 Else, curren version of this date ELSE PRESETS TEST.pd.zip

porres commented 4 weeks ago

not sure what I need to check in this patch, it has no toggles

porres commented 4 weeks ago

anyway, I guess I never tested this with toggles :)

the thing is that presets relies on [retrieve] sending a bang to collect the value of a GUI and the problem is that toggles change the state when receiving a bang. I don't think there is a way to fix it. What I can do is offer a toggle alternative that doesn't change the state when receiving a bang, well, I already to actually, it's called [button], but I'm not sure it works with presets either, but if it doesn't I can make it work!

pereshaped commented 4 weeks ago

[ELSE PRESETS TEST.pd.zip](https://github.com/user-attachments/files/15906828/ELSE.PRESETS.TEST.pd.zip

porres commented 4 weeks ago

if you really really want to use toggle, what you can do is this.

Screen Shot 2024-06-19 at 21 05 11

pereshaped commented 4 weeks ago

Alex I posted wrong patch. Just corrected. Hard day, it's late. Just sent the correct version just now. Good night for now

On Thu, 20 Jun 2024 at 02:03, porres @.***> wrote:

anyway, I guess I never tested this with toggles :)

the thing is that presets relies on [retrieve] sending a bang to collect the value of a GUI and the problem is that toggles change the state when receiving a bang. I don't think there is a way to fix it. What I can do is offer a toggle alternative that doesn't change the state when receiving a bang, well, I already to actually, it's called [button], but I'm not sure it works with presets either, but if it doesn't I can make it work!

— Reply to this email directly, view it on GitHub https://github.com/porres/pd-else/issues/2102#issuecomment-2179591511, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA7DGLS3YQOIBJYPKMB4ATLZIIL5HAVCNFSM6AAAAABJRVME72VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZZGU4TCNJRGE . You are receiving this because you authored the thread.Message ID: @.***>

porres commented 4 weeks ago

check my answers

pereshaped commented 4 weeks ago

Nice. thank you. I changed for radio buttons, so its ok right now.

Toggle is a can of worms anyway, so difficult to tame on any front

On Thu, 20 Jun 2024 at 02:06, porres @.***> wrote:

if you really really want to use toggle, what you can do is this.

Screen.Shot.2024-06-19.at.21.05.11.png (view on web) https://github.com/porres/pd-else/assets/5360142/0d86a370-6731-4ba8-9488-91de65275011

— Reply to this email directly, view it on GitHub https://github.com/porres/pd-else/issues/2102#issuecomment-2179593514, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA7DGLV5EMVJZAGSJGXXIX3ZIIMHBAVCNFSM6AAAAABJRVME72VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZZGU4TGNJRGQ . You are receiving this because you authored the thread.Message ID: @.***>

pereshaped commented 4 weeks ago

did, thanks. Good night

On Thu, 20 Jun 2024 at 02:09, porres @.***> wrote:

check my answers

— Reply to this email directly, view it on GitHub https://github.com/porres/pd-else/issues/2102#issuecomment-2179595461, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA7DGLTTUVCZSSAFNB2ZKODZIIMRVAVCNFSM6AAAAABJRVME72VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZZGU4TKNBWGE . You are receiving this because you authored the thread.Message ID: @.***>