Gui.cs: ComboBox assumes constructor initialization; needs to support object initialization

[tig]

If someone is using Computed Layout they have to use the only constructor provided:

    public ComboBox(int x, int y, int w, int h, IList<string> source)

Like this, which is gross:

        var comboBox = new ComboBox (0, 0, 0, 0, new List<string> () { " ~  s  gui.cs   master ↑10" } ) { 
            X = Pos.Right (label) + 1, 
            Y = Pos.Y (label), 
            Width = Dim.Percent (50),
             }

It also appears it was never tested with object initialziation because this causes it to crash:

        var comboBox = new ComboBox (0, 0, 0, 0, new List<string> () { " ~  s  gui.cs   master ↑10" } ) { 
            X = Pos.Right (label) + 1, 
            Y = Pos.Y (label), 
            Width = Dim.Percent (50),
                         Text = "foo",
             }

In general View classes should

1. Not require use of a parametrized constructor (except when forcing Absolute Layout *)
2. Avoid doing initialization via constructors. Instead use a property so consumers can use object initialization (e.g. var foo = new Foo() { a = b };).

** We should support specifying absolute coordinates via fields on View

[fergusonr]

@tig please assign this to me. I Have been looking at it, I'm about 70% done with the fix.

[tig]

@fergusonr Suggest you pull down #586 and #587 as you do this work.

[fergusonr]

Fix is done. PR #607