[Mono-dev] System.Web.UI/HtmlTextWriter.cs issue
kostat at mainsoft.com
Sun Jan 22 07:17:01 EST 2006
Please review the attached patch for semicolon.
From: mono-devel-list-bounces at lists.ximian.com
[mailto:mono-devel-list-bounces at lists.ximian.com] On Behalf Of Gonzalo
Sent: Friday, January 20, 2006 2:35 AM
To: mono-devel-list at lists.ximian.com
Subject: RE: [Mono-dev] System.Web.UI/HtmlTextWriter.cs issue
On Sun, 2006-01-15 at 07:35 -0800, Konstantin Triger wrote:
> Attached 2 patches, which, I believe, handle the issues better.
> 1. HtmlTextWriter.patch: addresses 1, 2 and 5.
> 2. button.patch:
> Point 3: My better understanding is that we must not delete user
> data. Thus we may only append our logic.
> Point 4: not in this particular case.
> Point 5: Unfortunately I do not know how to create a unit test
> for this scenario, as it depends on browser capabilities and some Page
> properties. Is there a way to control this?
The patch for HtmlTextWriter + test is ok to commit.
As for the button one... Can you do some manual tests to check which of
the user code and the system.web generated code go first in the onclick
attribute? As your patch is now, if there's no semicolon at the end, we
might get in trouble.
Mono-devel-list mailing list
Mono-devel-list at lists.ximian.com
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 747 bytes
Url : http://lists.ximian.com/pipermail/mono-devel-list/attachments/20060122/7b5e4459/attachment.obj
More information about the Mono-devel-list