ch7 treeview switch statement

jgitt7 · August 20th, 2013, 02:17 PM

Imar,

This is a great resource! Thanks for writing it. I'm having trouble with the switch statement that hides the Menu1 control and shows TreeView1. I coded it exactly as shown on pg. 259, but the DarkGrey theme still shows a menu control in FireFox. I also tried switching the "default" to

Code:

 Menu1.Visible= false; TreeView1.Visible = true;

but the browser still showed the menu. Monochrome has the menu as expected. I've included my source code below:

Code:

         <div id="MenuWrapper">
            <asp:Menu ID="Menu1" runat="server" CssClass="MainMenu" 
                DataSourceID="SiteMapDataSource1" Orientation="Horizontal" 
                StaticEnableDefaultPopOutImage="False">
            </asp:Menu>
            <asp:TreeView ID="TreeView1" runat="server" DataSourceID="SiteMapDataSource1" >
                    <LevelStyles>
                        <asp:TreeNodeStyle CssClass="FirstLevelMenuItems" />
                    </LevelStyles>
            </asp:TreeView>
            <asp:SiteMapDataSource ID="SiteMapDataSource1" runat="server" 
                ShowStartingNode="False" />
        </div>

Thank you!

Imar · August 20th, 2013, 02:24 PM

Hi there,

Good to hear you like the book.

Can you also post the code for the switch statement? And can you post the exact names of the Themes folders under App_Themes?

Cheers,

Imar

jgitt7 · August 20th, 2013, 02:28 PM

Switch statement:

Code:

             switch (Page.Theme.ToLower())
            {
                case "DarkGrey":
                    Menu1.Visible = false;
                    TreeView1.Visible = true;
                    break;
                default:
                    Menu1.Visible = true;
                    TreeView1.Visible = false;
                    break;
            }

My App_Themes folders are "DarkGrey" and "Monochrome".

I also forgot to add I'm using a web application, not a web site project. I commented out the parts of my Web.config file that I'm not using.

Imar · August 20th, 2013, 02:34 PM

Take a look at this:

Code:

switch (Page.Theme.ToLower())
{
    case "DarkGrey":
        Menu1.Visible = false;
        TreeView1.Visible = true;
        break;
    default:
        Menu1.Visible = true;
        TreeView1.Visible = false;
        break;
}

You're calling ToLower on the theme name, but then compare against a value that is not all lower case (DarkGrey). As such, that branch of the switch is never hit, and you always end up with a Menu and no TreeView.

Cheers,

Imar

jgitt7 · August 20th, 2013, 02:44 PM

It works now! Thank you.