[Mono-winforms-list] Patch for MessageBox and XplatUIX11

[Peter Dennis Bartok]

I will probably have to revert the patch. It seems that the win32 world has 
a different definiton of Modal.. The desired SetModal behaviour is that no 
other window can be selected instead of the one that's been made modal. The 
patch doesn't do that. (I tested it, but it seems I wasn't looking 
straight).

Peter

-----Original Message-----
From: "Peter Dennis Bartok" <peter@novonyx.com>
To: "Ashwin Bharambe" <ashwinb@gmail.com>; 
<mono-winforms-list@lists.ximian.com>
Date: 19 December, 2004 00:48
Subject: Re: [Mono-winforms-list] Patch for MessageBox and XplatUIX11


>Ashwin,
>
>Thanks for the patch. I've applied the XplatUIX11 part. However, there is a
>bigger problem with the MessageBox and Form code interaction, it currently
>creates two toplevel windows for the messagebox, I'd rather wait applying
>the patch that prevents the exception and unsets visible until I have that
>bug fixed.
>
>To answer your question: This wasn't too small a patch. I quite like 
>patches
>broken out by functionality, so they can be applied and tested independent
>of each other.
>
>Cheers,
> Peter
>
>-----Original Message-----
>From: "Ashwin Bharambe" <ashwinb@gmail.com>
>To: <mono-winforms-list@lists.ximian.com>
>Date: 18 December, 2004 18:22
>Subject: [Mono-winforms-list] Patch for MessageBox and XplatUIX11
>
>
>>Hi all,
>>
>>Attached is a patch for:
>>
>> - a small fix to MessageBox bug (setting Visible = false before
>>calling ShowDialog)
>> - adds modal support for dialog windows (XplatUIX11)
>>
>>As a result of this a simple MessageBox.Show("text", "caption") dialog
>>works fine on my linux box.
>>
>>Thanks,
>>Ashwin
>>
>>PS: Do let me know about the procedure and the granularity of sending
>>patches. I think this is probably too small a granularity for sending
>>patches.
>>
>
>_______________________________________________
>Mono-winforms-list maillist  -  Mono-winforms-list@lists.ximian.com
>http://lists.ximian.com/mailman/listinfo/mono-winforms-list
>
>