Use nameof(MyClass.MyProperty) in designer generated code for bindings

[tbolon]

Currently, the designer use the property names as string in the designer generated code.

It is really error prone when doing properties renaming, where the binding will fail only at runtime.

// 
// nameDataGridViewTextBoxColumn
// 
nameDataGridViewTextBoxColumn.DataPropertyName = "Name"; // <= this
nameDataGridViewTextBoxColumn.HeaderText = "Name";
nameDataGridViewTextBoxColumn.MinimumWidth = 8;
nameDataGridViewTextBoxColumn.Name = "nameDataGridViewTextBoxColumn";
nameDataGridViewTextBoxColumn.Width = 150;
// 
// textBox1
// 
textBox1.DataBindings.Add(new Binding("Text", personBindingSource, "Name", true)); // <= this
textBox1.Location = new Point(146, 368);
textBox1.Name = "textBox1";
textBox1.Size = new Size(150, 31);
textBox1.TabIndex = 1;

A better generated code will be:

// 
// nameDataGridViewTextBoxColumn
// 
nameDataGridViewTextBoxColumn.DataPropertyName = nameof(Person.Name); // <= this
nameDataGridViewTextBoxColumn.HeaderText = "Name";
nameDataGridViewTextBoxColumn.MinimumWidth = 8;
nameDataGridViewTextBoxColumn.Name = "nameDataGridViewTextBoxColumn";
nameDataGridViewTextBoxColumn.Width = 150;
// 
// textBox1
// 
textBox1.DataBindings.Add(new Binding("Text", personBindingSource, nameof(Person.Name), true)); // <= this
textBox1.Location = new Point(146, 368);
textBox1.Name = "textBox1";
textBox1.Size = new Size(150, 31);
textBox1.TabIndex = 1;

Note: The BindingSource already uses typeof(Person) in designer code, so there is already a way for the designer to know the full class name to use when reading/saving bindings.

Bonus: the new Binding("Text",...) should also use  nameof(TextBox.Name) statement.

[elachlan]

Could we also do textBox1.Name = nameof(textBox1);?