Radio buttons

[Steve Sierra]

Hi again,

I’ve come upon a new problem while learning UI building…

I’m just trying to have a simple comp created… but it seems my width and height variables block everything and I can’t figure out why. All the other variables work (the ignored line of code works fine).

Thanks for any help !
😉

{
function myScript(thisObj) {
function myScript_buildUI(thisObj) {
var myPanel = (thisObj instanceof Panel) ? thisObj : new Window("palette", "NEW COMP", [0, 0, 600, 600]);

res = "group{orientation:'column', alignment:['fill', 'fill'], alignChildren:['fill', 'fill'],\
compPanel: Panel{text:'Comp Settings', orientation:'column', alignChildren:['fill', 'fill'],\
nameGroup : Group{text:'Comp Name', orientation:'row', alignChildren:['left', 'fill'],\
cName: EditText{text:'New Comp'},\
cNameStaticText: StaticText{text:'Comp Name'},\
},\
widthGroup : Group{text:'Comp Width', orientation:'row', alignChildren:['left', 'fill'],\
cWidth: EditText{text:'1920'},\
cWidthStaticText: StaticText{text:'Comp Width'},\
},\
heightGroup : Group{text:'Comp Height', orientation:'row', alignChildren:['left', 'fill'],\
cHeight: EditText{text:'1080'},\
cHeightStaticText: StaticText{text:'Comp Height'},\
},\
pixelGroup : Group{text:'Comp PixAsp', orientation:'row', alignChildren:['left', 'fill'],\
cPA: EditText{text:'1'},\
cPAStaticText: StaticText{text:'Comp PixelAspect'},\
},\
durGroup : Group{text:'Comp Dur', orientation:'row', alignChildren:['left', 'fill'],\
cDur: EditText{text:'5'},\
cDurStaticText: StaticText{text:'Comp Duration'},\
},\
FPSGroup : Group{text:'Comp FPS', orientation:'row', alignChildren:['left', 'fill'],\
cFPS: EditText{text:'25'},\
cFPSStaticText: StaticText{text:'Comp FPS'},\
},\
},\
buttonGroup: Group{text:'Go', orientation:'column', alignChildren:['fill', 'fill'],\
myButton: Button{text:'Make New Comp'},\
},\
}"

// Adds resource string to panel
myPanel.grp = myPanel.add(res);

// Defaults

// Assign function to UI elements
myPanel.grp.buttonGroup.myButton.onClick = function (){

app.beginUndoGroup("NewComp");

var ncName = myPanel.grp.compPanel.nameGroup.cName.text;
var ncWidth = myPanel.grp.compPanel.widthGroup.cWidth.text;
var ncHeight = myPanel.grp.compPanel.heightGroup.cHeight.text;
var ncPixel = myPanel.grp.compPanel.pixelGroup.cPA.text;
var ncDuration = myPanel.grp.compPanel.durGroup.cDur.text;
var ncFPS = myPanel.grp.compPanel.FPSGroup.cFPS.text;

var newComp = app.project.items.addComp(ncName, ncWidth, ncHeight, ncPixel, ncDuration, ncFPS);
//var newComp = app.project.items.addComp(ncName, 1000, 1000, ncPixel, ncDuration, ncFPS);

app.endUndoGroup();
}
// Setup panel sizing and make panel resizable
myPanel.layout.layout(true);
myPanel.grp.minimumSize = myPanel.grp.size;
myPanel.layout.resize();
myPanel.onResizing = myPanel.onResize = function () {this.layout.resize();}

return myPanel;
}

// Build script panel
var myScriptPal = myScript_buildUI(thisObj);

if ((myScriptPal != null) && (myScriptPal instanceof Window)) {
myScriptPal.center();
myScriptPal.show();
}
}

// Execute script
myScript(this);
}


[Dan Ebberts]

I think you need to convert the width, height, pixelAspectRatio, duration and frameRate variables from text to numbers with parseInt() and/or parseFloat().

Dan

[Steve Sierra]

Hi Dan,

Thanks ! That works in deed.

I only need to parse width and height. It seems pixelAspectRatio, duration and frameRate are automatically read as numbers…
Is there a special rule for this ?

Is there a similar rule I am not applying with my radio buttons ?

Thanks again for your time.

Cheers !

[Steve Sierra]

Ok,

I found my error for the radio buttons… I had to redefine my variables for they were sliders and I had added other effects in the meantime.

So there was no error with the radio buttons ????

Cheers !