Add ButtonBarWidget class

[hhzl]

PolyWidget is a fat class. Factor out the control buttons in a separate class.

[hhzl]

It is not easy to see how the wiring of the GUI is done. Adding the buttons dynamically and then configuring them with a call to an Amber object method.

[sebastianconcept]

Do you think we can ease that with friendly comments in the code?

[hhzl]

OK, found method

renderButtonsOn: html

What I do not see easily where the method createWorld is called.

Poly calls PolyWidget new . Where does createWorld come into action?

[hhzl]

Friendly comments will be helpful.

[sebastianconcept]

commenting...

sebastian

o/

On 22/04/2014, at 16:49, Hannes Hirzel notifications@github.com wrote:

OK, found method

renderButtonsOn: html What I do not see easily where the method createWorld is called.

Poly calls PolyWidget new . Where does createWorld come into action?

— Reply to this email directly or view it on GitHub.

[hhzl]

What clutters the code is that there is an older version of the class embedded with methods like

• addStuffOld
• firstThing
• secondThing
• thirdThing
• addThing:

and maybe more. Correct?

[sebastianconcept]

I've pushed commented code and also cleaned dead code

I think all those are gone now.

When you can check and let me know so I can close this

[hhzl]

Thank you. It is easy to follow now. Thus it may serve as a guide how to integrate paperjs in Amber. This question was recently asked on the ML.

[hhzl]

In particular see http://juerglehni.com/works/scriptographer

[hhzl]

Are the methods in the protocols actions and accessing in the class Poly used at all?