MVC3 Tabs Helper - asp.net-mvc

Question

I need an html helper that would take care of "tabs" functionality on a page. clicking on the tabs will re-load the page and reload the partial view (if specified). I wrote it like so but not sure it this is the best solution.??
public static class TabExtensions
{
public static MvcHtmlString Tabs(this HtmlHelper htmlHelper, List<TabItem> tabItems, object htmlAttributes = null)
{
if (tabItems == null)
{
throw new ArgumentException("at least one tab item required");
}
string viewName = string.Empty;
object model = null;
var sb = new StringBuilder();
sb.Append("<a name=\"tabs\"></a>");
var tagUl = new TagBuilder("ul");
tagUl.MergeAttributes(HtmlHelper.AnonymousObjectToHtmlAttributes(htmlAttributes));
// Current url data
var baseUri = new UriBuilder(htmlHelper.ViewContext.HttpContext.Request.Url);
var selTab = htmlHelper.ViewContext.RequestContext.HttpContext.Request.QueryString["tab"];
foreach (var tab in tabItems)
{
// No tab user selected
if (string.IsNullOrEmpty(selTab))
{
selTab = tab.TabLinkText;
}
var tagLi = new TagBuilder("li");
string tagInnerHtml;
if (selTab.Equals(tab.TabLinkText, StringComparison.OrdinalIgnoreCase))
{
tagLi.MergeAttribute("class", "current");
tagInnerHtml = string.Format("<strong>{0}</strong>", tab.Text);
viewName = tab.PartialViewName;
model = tab.PartialViewModel;
}
else
{
tagInnerHtml = tab.Text;
}
var queryToAppend = string.Concat("tab=", tab.TabLinkText);
var querystring = new StringBuilder();
if (baseUri.Query.Length > 1)
{
if (baseUri.Query.Contains("tab"))
{
querystring.Append(baseUri.Query.Replace(string.Concat("tab=", selTab), queryToAppend));
}
else
{
querystring.Append(baseUri.Query + "&" + queryToAppend);
}
}
else
{
querystring.Append("?" + queryToAppend);
}
// Assign anchor link
querystring.Append("#tabs");
tagLi.InnerHtml = string.Format("{1}", querystring, tagInnerHtml);
tagUl.InnerHtml += tagLi.ToString();
}
sb.Append(tagUl.ToString());
// Render partial
if (!string.IsNullOrEmpty(viewName))
{
htmlHelper.ViewData.Model = model;
using (StringWriter sw = new StringWriter())
{
ViewEngineResult viewResult = ViewEngines.Engines.FindPartialView(htmlHelper.ViewContext.Controller.ControllerContext, viewName);
ViewContext viewContext = new ViewContext(htmlHelper.ViewContext.Controller.ControllerContext, viewResult.View, htmlHelper.ViewData, htmlHelper.ViewContext.TempData, sw);
viewResult.View.Render(viewContext, sw);
sb.Append(sw.GetStringBuilder().ToString());
}
}
return MvcHtmlString.Create(sb.ToString());
}
}
#region Tab Item Model
public class TabItem
{
public string Text { get; set; }
public string TabLinkText { get; set; }
public string PartialViewName { get; set; }
public object PartialViewModel { get; set; }
public TabItem(string text, string tabLinkText)
: this()
{
this.Text = text;
this.TabLinkText = tabLinkText;
}
public TabItem(string text, string tabLinkText, string partialViewName, object partialViewModel = null)
: this()
{
this.Text = text;
this.TabLinkText = tabLinkText;
this.PartialViewName = partialViewName;
this.PartialViewModel = partialViewModel;
}
public TabItem()
{
this.Text = string.Empty;
this.PartialViewName = string.Empty;
this.TabLinkText = string.Empty;
this.PartialViewModel = null;
}
}
#endregion
You use it like so:
<%
var tabList = new List<TabItem>
{
new TabItem(LocalResources.fld_AboutFirm_lbl, "about"),
new TabItem(LocalResources.fld_FirmOffers_lbl, "offer"),
new TabItem(LocalResources.fld_Profile_lbl, "profile", "~/Views/Partial/FirmProfileTab.cshtml", Model),
new TabItem(LocalResources.fld_Contact_lbl, "contact", "~/Views/Partial/FirmContactTab.cshtml", Model)
};
%>
<%: Html.Tabs(tabList, new { #class = "firmTabs clearfix" })%>
this will generate html:
<ul class="firmTabs clearfix"><li>O firmie</li><li>Firma oferuje</li><li>Profil</li><li class="current"><strong>Kontakt</strong></li></ul>

Answer 1

I think the better solution is to make master page for tab menu and view pages for tab content. Your approach looks very complex for me. Why do you need html helper for this? If you encapsulate your html into helper method - you loose your view. So in terms of MVC your way is not good I think.