<html><body><div style="color:#000; background-color:#fff; font-family:times new roman, new york, times, serif;font-size:12pt"><div><span>Argh...one more dumb oversight in my change.</span></div><div><span>Enclosed is ANOTHER version of the patch.</span></div><div><span>I wish I had the luxury of working on my hobbies when I was awake and energetic. ;-)</span></div><div><span><br></span></div><div><span>Steven Boswell</span></div><div><br></div>  <div style="font-size: 12pt; font-family: 'times new roman', 'new york', times, serif; "> <div style="font-size: 12pt; font-family: 'times new roman', 'new york', times, serif; "> <div dir="ltr"> <font size="2" face="Arial"> <hr size="1">  <b><span style="font-weight:bold;">From:</span></b> Steven Boswell II <ulatekh@yahoo.com><br> <b><span style="font-weight: bold;">To:</span></b> "mono-devel-list@lists.ximian.com" <mono-devel-list@lists.ximian.com> <br> <b><span style="font-weight:
 bold;">Sent:</span></b> Saturday, June 2, 2012 7:58 PM<br> <b><span style="font-weight: bold;">Subject:</span></b> Re: [Mono-dev] Patches for mono-winforms<br> </font> </div> <br>
<div id="yiv1647911984"><div><div style="color: rgb(0, 0, 0); background-color: rgb(255, 255, 255); font-size: 12pt; font-family: 'times new roman', 'new york', times, serif; "><div>Rob, you're my hero.  Very few tests in <a target="_blank" href="http://DataGridViewTest.cs">DataGridViewTest.cs</a> create a Form, but most of the ones that do involve data binding.</div><div>I added a Form to my test, and it succeeded immediately.  Apparently, Application.Run() isn't necessary, but that was a good idea.</div><div>Enclosed is a revised patch, for review by the Powers That Be.</div><div><br></div><div>Steven Boswell</div><div><br></div>  <div style="font-size: 12pt; font-family: times, serif; "> <div style="font-size: 12pt; font-family: times, serif; "> <div dir="ltr"> <font size="2" face="Arial"> <hr size="1">  <b><span style="font-weight:bold;">From:</span></b> Rob Wilkens <robwilkens@gmail.com><br> <b><span
 style="font-weight:bold;">To:</span></b> mono-devel-list@lists.ximian.com
 <br> <b><span style="font-weight:bold;">Sent:</span></b> Saturday, June 2, 2012 6:46 PM<br> <b><span style="font-weight:bold;">Subject:</span></b> Re: [Mono-dev] Patches for mono-winforms<br> </font> </div> <br>
<div id="yiv1647911984">
  

    
  
  <div>
    On 06/02/2012 09:38 PM, Rob Wilkens wrote:
    <blockquote type="cite">

      
      On 06/02/2012 08:55 PM, Steven Boswell II wrote:
      <blockquote type="cite">
        <div style="color: rgb(0, 0, 0); background-color: rgb(255, 255, 255); font-size: 12pt; font-family: times, serif; ">
          <div><span>The EditingControlShowing event has to be called,
              and it has to be called after the control's contents have
              been initialized properly...that's not really two separate
              issues.</span></div>
          <div><span><br>
            </span></div>
          <div><span>The enclosed patch is an updated version; in
              addition to having a unit test, it fixes one additional
              bug revealed by my testing. Before,
              DataGridViewComboBoxCell.InitializeEditingControl() was
              setting the initial value from the FormattedValue
              property, instead of the initialFormattedValue parameter.</span></div>
          <div><span><br>
            </span></div>
          <div><span>I tried to write an additional unit test that
              worked with bound data, but for the life of me I can't
              figure out why it doesn't work.  I've done data-binding
              with DataGridView before...it wasn't this mysterious.  In
              my unit test, after I set the DataGridView's DataSource
              property, the data-grid doesn't initialize properly;
              instead of four rows, it ends up with one row, and all its
              cell values are null.  After beating my head against the
              wall for several hours, I'm perfectly happy to be told
              what a moron I am, if someone will just tell me why
              the EditingControlShowingTest_Bound test doesn't work. :-)</span></div>
          <div><span><br>
            </span></div>
          <div><span>Steven Boswell</span></div>
          <br>
        </div>
      </blockquote>
      <br>
      Not writing to call you a moron, I'm a newbie myself and i could
      be wrong...  But i copied and pasted your test code for
      EditingControlShowingTest_Bound into Visual Studio 2010, but
      rather than creating the DataGridVIew in code i placed it on the
      form and modified your code to use the one on the form...  And i
      disabled the asserts...  And from what i can tell it runs fine in
      both .net and mono (that is, the data grid view populates).<br>
      <br>
      One thing that I noticed about your code, though, is it depends on
      a 'showing' event..<br>
      <br>
      And i wonder if that means your datagridview needs to be placed on
      a form which is displayed, so that it is actually shown...<br>
      <br>
      i.e. change your code something like this:<br>
      <br>
      using (Form Form1=new Form()){<br>
          Form1.Controls.Add(_dataGridView);//optionaly set sizeand
      location of both gridview and form<br>
          Form1.Show();<br>
      <br>
          ... insert the rest of your code here .... <br>
      }<br>
    </blockquote>
    Oh, and if i'm right, you might need the equivalent of an
    Application.Run(Form1) to process the events since you're listening
    for events, just don't forget to close Form1 or the Run loop might
    never end and your test can hang up with a displayed window...<br>
    <br>
    <br>
  </div>

</div><br>_______________________________________________<br>Mono-devel-list mailing list<br><a rel="nofollow" ymailto="mailto:Mono-devel-list@lists.ximian.com" target="_blank" href="mailto:Mono-devel-list@lists.ximian.com">Mono-devel-list@lists.ximian.com</a><br>http://lists.ximian.com/mailman/listinfo/mono-devel-list<br><br><br> </div> </div>  </div></div></div><br>_______________________________________________<br>Mono-devel-list mailing list<br><a ymailto="mailto:Mono-devel-list@lists.ximian.com" href="mailto:Mono-devel-list@lists.ximian.com">Mono-devel-list@lists.ximian.com</a><br><a href="http://lists.ximian.com/mailman/listinfo/mono-devel-list" target="_blank">http://lists.ximian.com/mailman/listinfo/mono-devel-list</a><br><br><br> </div> </div>  </div></body></html>