Script panel label keyboard entry?

Christoph Hart

@Christoph-Hart In the meantime you can just set parentComponent to an empty string, then it won't traverse the component list where it's currently crashing...

d.healey

I remove the parent property (I missed that before!) and now clicking the button no longer causes a crash, but no popup appears either.

Lindon

@Christoph-Hart so if we are here in UI widget property land - can I re-raise my feature request for using an array of values for the pop-up display in a slider?

So simply put:

  {
    "type": "ScriptSlider",
    "id": "Knob1",
    "x": 114.0,
    "y": 65.0,
    "style": "Knob",
    "middlePosition": 3,
    "filmstripImage": "Use default skin",
    "linkedTo": "",
    "max": 5.0,
    "min": 1.0,
    "displayValues":["red","green","yellow", "blue", "orange"]
  }

so if displayValues is blank (empty array), it would display as it does now....for reference the way knobMan does this is pretty cool....

Christoph Hart

@Lindon you already have a getValuePopupText function so this would just duplicate the functionality with less flexibility.

Lindon

@Christoph-Hart said in Script panel label keyboard entry?:

@Lindon you already have a getValuePopupText function so this would just duplicate the functionality with less flexibility.

er well....

I cant see any documentation on the web for this call
There's no inline documentation in the script editor
when I call this

const var SoundVol3 = Content.getComponent("SoundVol3");

SoundVol3.getValuePopupText();

it tells me Interface:! StdControls.js (54): function not found

I'm not trying to "get" anything... I'm trying to make it display these words(colours here) when I move the knob...

An example

I have dynamically assigned knobs for a range of effects, for one effect at least one of these knobs controls a setting who's values are: 3,4,5,8,10,12,16 - so I want a way to set the control to
min= 1,
max = 7,
displayValues = [3,4,5,8,10,12,16]

d.healey

@Lindon I think it's this. Set rather than get

Christoph Hart

@d-healey yup I meant that one.

Lindon

@d-healey said in Script panel label keyboard entry?:

@Lindon I think it's this. Set rather than get

thanks but this is very ugly now...
so given my example I need to (psudo-code):

if loaded_FX == XXXX
{
      if control id == YYYY
           control.setValuePopupFunction(myFunc(dbl){
                Switch dbl
                      case 1:
                            return "3";
                      case 2:
                            return "4";
                   etc. etc
                      case 7:
                            return "16";
          }
}

which seems very klunky - -- hey ho.

Christoph Hart

@Lindon it‘s only ugly if you code it as ugly as your example. I can guarantee you that it will become a one-liner if you setup your data model right.

You need a way of customizing the value popup and the most flexible way is to define a function that returns a string.

Lindon

@Christoph-Hart said in Script panel label keyboard entry?:

@Lindon it‘s only ugly if you code it as ugly as your example. I can guarantee you that it will become a one-liner if you setup your data model right.

You need a way of customizing the value popup and the most flexible way is to define a function that returns a string.

but for 95% of the time I dont need to do this - so I can have this happen - in 1 line if you like - but then how do i make it UN-happen for my next slot loaded effect?

Christoph Hart

@Lindon just call the function with a non function object as parameter (empty string or whatever).

Lindon

@Christoph-Hart said in Script panel label keyboard entry?:

@Lindon just call the function with a non function object as parameter (empty string or whatever).

so:

control.setValuePopupFunction(myFunc(""){});

??

Christoph Hart

control.setValuePopupFunction(““)

Lindon

@Christoph-Hart said in Script panel label keyboard entry?:

control.setValuePopupFunction(““)

..and this will then use the default "normal" values?

..and can we get the documentation updated to reflect this "clear my function" stuff?

aaronventure

@Christoph-Hart Looks like CHRISmas came early.

Cheers, funky.

I did have it hacked together with the method I described earlier and the keyPressCallback to hide the label again, but this is so much cleaner and less prone to any buggy behavior (also it can take alignment, so I can actually have that pink background I always wanted), with plenty of other use cases as well beyond what I wanted.

Got it fully encapsulated with no issues. Happy to be able to pass relative parameters to it so that it adjusts to whichever control triggers it.

Christoph Hart

Alright, took another spin at it and I found the most embarrassing code typo ever that caused an infinite loop and crash when using the parentComponent property. Should be fixed now.

Here's a snippet that lets you open a text box for sliders (as soon as you set the sliderStyle property to "Horizontal" / "Vertical" you'll loose the text box so this brings it back with even more customization options:

const var textInputBroadcaster = Engine.createBroadcaster({
	"id": "text-input",
	"args": ["component", "value"],
	"tags": ["text-input"]
});

const var List = [Content.getComponent("Knob5"),
                  Content.getComponent("Knob4"),
                  Content.getComponent("Knob3"),
                  Content.getComponent("Knob2"),
                  Content.getComponent("Knob1")];

textInputBroadcaster.attachToComponentMouseEvents(List, "Clicks Only", "");

textInputBroadcaster.addListener("", "show textbox", function(component, event)
{
	if(event.shiftDown && !event.mouseUp)
	{
		var obj = {
			"text": Engine.doubleToString(component.getValue(), parseInt(Math.log10(component.get("stepSize")) * -1)) + " " + component.get("suffix"),
			"x": component.getGlobalPositionX() + component.get("width") / 2 - 30,
			"y": component.getGlobalPositionY() + component.get("height") / 2 - 12,
			"width": 60,
			"height": 24
		};
		
		var c = component;
		
		Content.showModalTextInput(obj, function[c](ok, input)
		{
			if(ok)
			{
				c.setValue(parseFloat(input));
				c.changed();
			}
		});
	}
});

d.healey

I still don't see a text box.

d.healey

@d-healey Hmm if I comment out the call to grabKeyboardFocus() I don't get an error but I still don't get an input box.

aaronventure

@Christoph-Hart said in Script panel label keyboard entry?:

I found the most embarrassing code typo ever that caused an infinite loop and crash when using the parentComponent property

lmao

happens to the best of us

Christoph Hart

@d-healey how does this happen? I don't hit that assertion, but it seems that it tries to add the text input to a component that isn't visible yet.

The entire system was not trivial to implement because you have to send a notification from the scripting thread to the UI which resides on a completely different data model, but I think I got all edge cases sorted out with the last commit (and it's working fine here).

Script panel label keyboard entry?

20

1.7k

11.8k

102.7k