Suggestion for things such as SiteHome.OnPreInit() [/web/default.aspx.cs] SiteMapLink.Render()[/web/controls/SiteMapLink.cs] and PageMenu.RenderTreeView() [/web/controls/PageMenu.ascx.cs]:

Any control/class that writes out links to the screen (style sheets, sitemap links, etc) should have a property to set the link(s) to write.  For example: 

Currently, the code for stylesheets looks like so:

[code]

<portal:StyleSheet id="StyleSheet" runat="server" />

StyleSheet stylesheet = (StyleSheet)Master.FindControl("StyleSheet");
if (stylesheet != null)
{
string baseSkinUrl = SiteUtils.GetSkinBaseUrl(allowPageOverride);

stylesheet.LiteralStyleSheetUrl.Text
= "\n<link href='"
+ baseSkinUrl
+ "style.css' type='text/css' rel='stylesheet' />"
+ "\n<link href='"
+ baseSkinUrl
+ "styletext.css' type='text/css' rel='stylesheet' title='" + Resource.NormalTextLabel + "' />"
+ "\n<link href='"
+ baseSkinUrl
+ "styletextmedium.css' type='text/css' rel='alternate stylesheet' title='" + Resource.MediumTextLabel + "' />"
+ "\n<link href='"
+ baseSkinUrl
+ "styletextlarge.css' type='text/css' rel='alternate stylesheet' title='" + Resource.LargeTextLabel + "' />"
+ "\n<link href='"
+ baseSkinUrl
+ "styleprinter.css' type='text/css' rel='stylesheet' media='print' />";
}

[/code]

I suggest that it be changed to something like this:

[code]

<portal:StyleSheet id="StyleSheet" runat="server" files="style.css,styletext.css"/>

StyleSheet stylesheet = (StyleSheet)Master.FindControl("StyleSheet");

string[] files;
if (stylesheet != null)
{
string baseSkinUrl = SiteUtils.GetSkinBaseUrl(allowPageOverride);
files = stylesheet.Files.split(",");

for(int i = 0; i < files.length; i++){
stylesheet.LiteralStyleSheetUrl.Text += "\n<link href='"
+ baseSkinUrl + files[i] + "'"
+ type='text/css' rel='stylesheet' title='" + Resource.NormalTextLabel + "' />"

}

}

[/code]