Arranging Lights in Lightmix

[Frood]

I also had the delete button in mind but thought it would be more consistent if adding and deleting stays in render setup because to include adding would be quite complex :) I like the multi selection and direct rename feature very much.

But I miss a default icon definition and fn clamp. It is undefined and I get the usual error when trying to d&d.


Good Luck

[Avi]

I hope this fixes the issue.

[Frood]

In fact there were further issues. I see that additionally the rtFil handler is already fixed, now there is still is a non functional callback to auto refresh the list. Callback functions have to be global unfortunately because they use another scope, so "REO_roll != undefined" will always be false, and you cannot call the required functions on it. But just add a "global REO_roll" (yes, we all hate globals...) before defining the rollout and the auto refresh will work.

Then: please update the version variable to 0.4, not to 4.0 :) And the button state (pressed) when the script is displayed (on isChecked()) is gone as well as the docking option. And I'm no friend of fixed layouts.

But those are features I want to have for myself primarily and do not affect functionality. With the exception of removing the fixed layout where I struggle when it comes to .net controls, I have brought back those features into your previous version here for personal use :)


Good Luck

[Avi]

Hi, I have revised most of the things but few things I still cannot solve. #buttonState (It only updates when some other max UI is refreshed) and I have not touched layout flexibility, still it uses fixed size (will play with it later) Version update is fixed, docking functionality is brought back :) Global Reo_roll declaration fixed. #fixed colors too so its blends easily with max interface

Feel free to improve it further.

[Frood]

Nice :)

My comments after quickly looking at it:

(1) As for button state: replace line 195 to 198 by just 'updatetoolbarbuttons()'. Usually max calls this itself at every reasonable and stupid occasion (for example when just selecting something in scene explorer), but sometimes fails when it would be important. Same at line 305 to 308.

(2) Kick line 293 and 302 - it does not work like this, 'buttonText' is no variable but part of the macro definition.

(3) 'useCUI' is 'true' initially, leading to an error (catched by try/catch) in 'on execute' if you press the button when the window is open because 'unRegisterDialogBar()' is executed without having the dialog registered. Has to be defined as 'false' initially.


Good Luck

[Avi]

Thanks for the feedback. Now it works well.

[CharlyRT]

Hi
I was trying to add 2 buttons from v0.3
Move UP and Down
I used the code from that version but it seems thar the whole code change in v0.4 and I cant use them.
Could it be possible to help on this one

Im attaching what I was trying to... obviusly I dont know anything about of scripting :(