Ability to move sampler / synth / container modules around

[aaronventure]

@d-healey That's true. But the realtime onTimer callback runs in seconds, the and the timer objects in milliseconds so I won't pretend to know how the sausage is made and what goes on in the meatshop behind the counter

[Christoph Hart]

Alright guys, here we go:

- added drag&drop of modules within the patch browser! · christophhart/HISE@888851e

The open source framework for sample based instruments - - added drag&drop of modules within the patch browser! · christophhart/HISE@888851e

GitHub (github.com)

I tried to cover a few common edge cases, but I'm sure there's lots to discover when doing monkey testing, so let me know if you find glitches...

[d.healey]

@Christoph-Hart Excellent!

[aaronventure]

christophhart Already? I can't imagine what we're getting for Christmas, then!

[gorangrooves]

@Christoph-Hart wow. Bravo! That looks great. This is a serious UX improvement. Thank you so much!

[Dan Korneff]

@Christoph-Hart
Getting a couple build error on MacOS

HISE/hi_backend/backend/debug_components/PatchBrowser.cpp:1512:23 Ambiguous conversion from derived class 'hise::PatchBrowser::PatchItem' to base class 'hise::Processor::BypassListener':
    class hise::PatchBrowser::PatchItem -> class hise::PatchBrowser::ModuleDragTarget -> class Processor::BypassListener
    class hise::PatchBrowser::PatchItem -> class Processor::BypassListener

HISE/hi_backend/backend/debug_components/PatchBrowser.cpp:1548:40 Ambiguous conversion from derived class 'hise::PatchBrowser::PatchItem' to base class 'hise::Processor::BypassListener':
    class hise::PatchBrowser::PatchItem -> class hise::PatchBrowser::ModuleDragTarget -> class Processor::BypassListener
    class hise::PatchBrowser::PatchItem -> class Processor::BypassListener

[Matt_SF]

@Christoph-Hart rules

[tomekslesicki]

@Christoph-Hart this is an awesome quality of life improvement! I'm on the latest Develop build but I can't see any of the extra stuff - there's no + and bin icons, and I still can't drag-and-drop the modules around. Is there something I need to enable to get this or is it another branch or what? ;-)

[Straticah]

@tomekslesicki did you go into edit mode first using the pen? ( i know it is a little bit confusing)
it would make sense to have the drag /re arrange option in both modes tho imo

[Christoph Hart]

it would make sense to have the drag /re arrange option in both modes tho imo

No I think this will lead to moving modules around accidentally too often, also from a UX standpoint I think it makes sense to restrict moving (aka editing the graph) to the edit mode.

[Straticah]

@Christoph-Hart yeah maybe, but people are kind of used to a certain workflow and i am not aware of any other "layer based" software that does not support dragging on the layer view.

I use bitwig, ableton, photoshop, figma, adobeXD, Illustrator, Blender and Cinema4D and they all have this layer drag behaviour.

But they are creative tools (and maybe not comparable to HISE) but they definitely are highly workflow optimized and therefore simplified from a UX standpoint.

[tomekslesicki]

@Straticah ah, no, I didn’t know this was required! I’ll look for that pen tomorrow when I’m back at the studo, thank you :-) Is it somewhere close to the graph or hidden in some smart place @Christoph-Hart ?

[Straticah]

@tomekslesicki

[gorangrooves]

@Straticah @Christoph-Hart Either way works equally well when you are aware of how to use it. But, being able to do it without unlocking first is more intuitive. I do see a point in preventing accidental moving. I just wonder how easy and likely it would be to accidentally move it...

[tomekslesicki]

@Christoph-Hart I agree, this should be the default interaction mode. It's very easy to click on the modules, I have not had a single accident when I dragged something around not intended.

[Straticah]

@tomekslesicki i can imagine giving the components a bigger height might work aswell and is less fiddly. At least in the default view where quite a lot of vertical space is unused.