Re: [WARNING : A/V UNSCANNABLE] RE: [Mono-dev] System.Web.UI.Page patch

2006-04-25 Thread Chris Toshok
On Tue, 2006-04-25 at 02:16 -0700, Andrew Skiba wrote: I'd like to see the test used for this as well. Did you see the standalone test that I sent yesterday? For convenience, I attach it to this message, too. It covers the following patches: ApplyTheme.patch Page.Web.Config.patch

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

2006-04-24 Thread Chris Toshok
I'd like to see the test used for this as well. At first blush the theme change looks ok, but I really don't like the idea of adding the ApplyTheme behavior to OnInit. Unless, of course, this is what actually happens in MS's implementation. Also, the master page change doesn't seem necessary to

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

2006-04-23 Thread 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