[Mono-dev] System.Web.UI/HtmlTextWriter.cs issue

[Konstantin Triger]

Hi Gonzalo,

I ensured that the user code goes first. Regarding the semicolon: I
checked on net1.1, - it's not added there. Now I checked on net2.0, - it
is appended there. So I'll add it for both platforms as this seems a
regular bug fix.

If someone objects on adding the semicolon for net_1_1, please reply.

Regards,
Konstantin Triger


-----Original Message-----
From: mono-devel-list-bounces at lists.ximian.com
[mailto:mono-devel-list-bounces at lists.ximian.com] On Behalf Of Gonzalo
Paniagua Javier
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.

-Gonzalo


_______________________________________________
Mono-devel-list mailing list
Mono-devel-list at lists.ximian.com
http://lists.ximian.com/mailman/listinfo/mono-devel-list