[Mono-dev] System.Web.UI.Page patch

[Andrew Skiba]

Please ignore the previous patch, review the new patch instead. Sorry
for the inconvenience.

The previous patch did not handle the situation when StyleSheetTheme
should affect a control created by template (as ImageButton inside
LoginView).

To find out the correct flow, I subclassed an ImageButton in dotnet, and
watched the order in which the following methods were called:

EnsureChildControls, OnPreRender, OnLoad, OnInit, ApplyStyleSheetSkin,
set_ImageUrl

In current mono implementation theme is applied for all the control
hierarchy before applying the master page. But at this stage many
controls still don't exist. My previous patch solved problem with master
page, but not with contols created during PreRerderRecursiveInternal.
This patch solves the problem and removes code duplication for
recursion.

If no one objects, I will commit.

Thank you.
Andrew.

> -----Original Message-----
> From: mono-devel-list-bounces at lists.ximian.com 
> [mailto:mono-devel-list-bounces at lists.ximian.com] On Behalf 
> Of Andrew Skiba
> Sent: Sunday, April 23, 2006 11:46
> To: mono-devel-list at lists.ximian.com
> Subject: [Mono-dev] System.Web.UI.Page patch
> 
> The attached patch fixes the flow for themes combined with 
> master pages.
> If no one objects, I will commit.
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ApplyTheme.patch
Type: application/octet-stream
Size: 2011 bytes
Desc: ApplyTheme.patch
Url : http://lists.ximian.com/pipermail/mono-devel-list/attachments/20060423/13feda5b/attachment.obj