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
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
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