Address TODO about propagating options to button model.

[pixelzoom]

This NOTE was added by @jonathanolson when he converted RectangularPushButton to TypeScript. It should have been a TODO, with an issue. The same problem needs to be addressed in RoundPushButton, and any other sun button that propagates all of its options to its model class. It's unlikely that there's a TODO in every location where this needs to be addressed.

// NOTE: Let's not create PushButtonModel with these options?
export type RectangularPushButtonOptions = SelfOptions & RectangularButtonOptions & PushButtonModelOptions;

[samreid]

It's not really clear what the TODO means here, @pixelzoom or @jonathanolson can you clarify?

[pixelzoom]

Passing the same options to super and a subcomponent is an anti pattern. For example, superOptions in RectangularPushButton:

50     const pushButtonModel = new PushButtonModel( superOptions );
51 
52     super( pushButtonModel, new PushButtonInteractionStateProperty( pushButtonModel ), superOptions );