Hjælp til menu i user control

hehe... ja, det kan godt gøres noget pænere ved hjælp af WebControls... du kan enten bruger HyperLink som svarer til et <a>-tag og Label som svarer til <span>.

Begge disse typer arver fra en base-control som implementerer forskellige attributer, heriblandt en CssClass og en Style hvor du kan tilføje forskellige styles.

For at udnytte det i din repeater skal du ændre din signatur på din metode så den returnerer et object af typen System.Web.UI.WebControls.WebControl, og dine to strenge skal ændres til

1)

Label l = new Label();
l.CssClass = "menu_Alm";
l.Style.Add("color", "red");
l.Text = menunavn.ToString();
return l;

2)

HyperLink hl = new HyperLink();
hl.NavigateUrl = filnavn.ToString();
hl.CssClass = "menu_Alm";
hl.Text = menunavn.ToString();
return hl;

Når jeg gør som du har beskrevet, skriver den navnet på webcontrollen i menuen;

eks.
System.Web.UI.WebControls.Label
System.Web.UI.WebControls.HyperLink
System.Web.UI.WebControls.HyperLink
System.Web.UI.WebControls.HyperLink

Hard et noget at gøre med det tag jeg bruger til at præsentere data i repeateren:
<%# valgtPunkt(DataBinder.Eval(Container.DataItem, "modulNr"), DataBinder.Eval(Container.DataItem, "filnavn"), DataBinder.Eval(Container.DataItem, "menunavn"))%>

hm.. ja, det er da også rigtigt nok... kan være vi skal fuske lidt så *smiler skummelt*

prøv at ændr din signatur af din metode tilbage til string. Istedet for at smide objectet retur renderer vi det bare og sender html'en tilbage:

1)

1)

Label l = new Label();
l.CssClass = "menu_Alm";
l.Style.Add("color", "red");
l.Text = menunavn.ToString();

StringWriter sw = new StringWriter();
HtmlTextWriter htmlTextWriter = new HtmlTextWriter(sw);
l.Render(htmlTextWriter );
return sw.ToString();

og,ja... mon ikke du kan gætte 2'eren selv? :)


det er dog noget af en hovsaløsning. Istedet kan man sætte en placeholder i din template og override ItemDataBound der laver et af de to objecter og sætter det ind i placeholderen.

SÅ er det måske lige før at den første løsning er lige så pæn.

Hvad med hensyn til det at teksten skifter farve når man køre musen henover?

tja... det kan der være noget om... eneste ulempe ved din metode er at du ikke udnytter de indbyggede klasser til formålet i .Net. Det er bare lidt noget bæh at bruge ItemTemplate i DataReaderen til at udskrive menuen på, brug istedet ItemDataBound-eventen, hvor du kan få lov at lave noget conditional casting af dit object så det passer til den type der kommer fra valgtPunkt-metoden.

ang. at skrifte fave skal du bruge a:hover i css

Den virker ikke i min usercontrol!

Jeg fik iøvrigt ikke det her til at virke:

l.Render(htmlTextWriter );

Den brokker sig over "Render()"

hm... det skal virke... :hover altså... http://www.webstartcenter.com/howto/css_links.php

Ang. det brok... hvilken fejl får du?

Nu fatter jeg ikke en lyd...

Nu virker det, det er som om at den ikke rebuilder all når jeg beder om det! Nogle gange hvis jeg laver retur typen om på en metode slår det ikke igennem før jeg har makket rudt med det. Hvad kan det skyldes?

Nå men nu hvor det virker, så smid et svar så du kan få nogle velfotjente point!

Iøvrigt hvis du får tid til det kan du så give et eksempel på  den "mere rigtige" løsnig du referer til ? (Istedet kan man sætte en placeholder i din template og override ItemDataBound der laver et af de to objecter og sætter det ind i placeholderen.)

hm, det ved jeg ikke lige, hvorfor at den skal være så tvær :/

men jo, jeg kan da lige komme med et eksempel.

Tusind tak for din hjælp!

Glæder mig til at se dit eksempel (det haster ikke)

noget i den retning

<asp:repeater id="rptProducts" runat="server">

    <ItemTemplate>
        <asp:PlaceHolder id="menuItem" runat="server" />
    </ItemTemplate>

</asp:repeater>

override protected void OnInit(EventArgs e)
{
    this.rptProducts.ItemDataBound += new RepeaterItemEventHandler(rptProducts_ItemDataBound);

    base.OnInit(e);
}

private void rptProducts_ItemDataBound(object sender, RepeaterItemEventArgs e)
        {
            if (e.Item.ItemType == ListItemType.Item || e.Item.ItemType == ListItemType.AlternatingItem)
            {
                System.Data.OleDb.OleDbDataReader reader = (System.Data.OleDb.OleDbDataReader)e.Item.DataItem;
                int modelNr = (int)reader["modelNr"];
                string filNavn = (string)reader["filNavn"];
                string menuNavn = (string)reader["menuNavn"];
               
                PlaceHolder menuItem = e.Item.FindControl("menuItem");

                Control c = valgtPunkt(modelNr, filNavn, menuNavn);
                Label l;
                HyperLink hl;

                if (typeof(c) == typeof(Label))
                {
                    l = (Label)c;
                    menuItem.Controls.Add(l);
                }
                else if (typeof(c) == typeof(HyperLink))
                {
                    hl = (HyperLink)c;
                    menuItem.Controls.Add(hl);
                }
            }
        }

eneste ting jeg er lidt usikker på er denne linje

System.Data.OleDb.OleDbDataReader reader = (System.Data.OleDb.OleDbDataReader)e.Item.DataItem;

da jeg ikke ved hvilken type du har brugt til at databinde til din repeater

og egentlig, så skulle den casting slet ikke være nødvendigt... dette burde kunne bruges

private void rptProducts_ItemDataBound(object sender, RepeaterItemEventArgs e)
        {
            if (e.Item.ItemType == ListItemType.Item || e.Item.ItemType == ListItemType.AlternatingItem)
            {
                System.Data.OleDb.OleDbDataReader reader = (System.Data.OleDb.OleDbDataReader)e.Item.DataItem;
                int modelNr = (int)reader["modelNr"];
                string filNavn = (string)reader["filNavn"];
                string menuNavn = (string)reader["menuNavn"];
               
                PlaceHolder menuItem = e.Item.FindControl("menuItem");

                Control c = valgtPunkt(modelNr, filNavn, menuNavn);
                menuItem.Controls.Add(c);
            }
        }