helgoboss / helgobox

Helgobox: ReaLearn & Playtime
https://www.helgoboss.org/projects/helgobox
GNU General Public License v3.0
205 stars 20 forks source link

FR: Enhancement: Streamlining FX naming conventions for {{target.fx.name}} #772

Open PucksEvilTwin opened 1 year ago

PucksEvilTwin commented 1 year ago

This placeholder could be improved quite a bit with some regex before it reaches the control surface. With minimal available characters on displays, it would be a good idea to make the following wholesale changes:

  1. Cut out any FX type prefix. VST, VST3, JS etc. so that VST3: Lindell SBC = Lindell SBC (so anything before ": ")
  2. Cut out anything after "(" these are normally just manufacturers names.
helgoboss commented 1 year ago

Makes sense. However, I wouldn't change the content of target.fx.name itself - it just reflects whatever name the FX instance has, so it can be also a custom user-defined name. Instead, I would introduce new placeholders that reflect the original plug-in name. First draft:

Any feedback appreciated. Maybe there's a better wording for type but I can't come up with one.

PucksEvilTwin commented 1 year ago

I suppose my suggestion would be to drop ".type" all together. I think they all make sense without that.

If that messes with target.fx.name, maybe make our new placeholder to target.fx.displayname as it would be optimized for displays.

Sorry if that's unhelpful. I'm terrible at coming up with syntax :)

helgoboss commented 1 year ago

Yeah, maybe the .type is a bit more qualifying than necessary. In this case, I would go for target.fx.original_name (instead of target.fx.type.full_name) and target.fx.original_short_name (instead of target.fx.type.name).