Hi,
I've made the change you indicated in SiteUtils.DetermineSkinBaseUrl(string skinName)
However note that the only reason that code is executed is because you have
<add key="CombineCSS" value="false"/>
and there is really no good reason to do that, it makes it create separate links for each css file which is bad for performance and there is no control over caching when you do that because the browser will cache static css files.
For supporting pages of a CMS plugin feature it is easy (with the latest code) to make the supporting page use the page specific skin (ie the skin selected in page settings on the cms page) like this:
protected override void OnPreInit(EventArgs e)
{
AllowSkinOverride = true;
base.OnPreInit(e);
}
If your page is not a supporting page of a cms plugin or if you are trying to hard code a specific skin I don't really have any advice for you about "best practices", whatever works for you is ok with me but it doesn't sound like something I would do myself.
Hope that helps,
Joe