Maybe you're not doing anything wrong; it just depends on what you are doing. I noticed this:
Code:
[list]
<li id="video" runat="server" Visible='<%# Eval("iVideoId") IsNot DBNull.Value %>'>
HyperLink here
</li>
</ul>
With this setup, you repeat the[list] element for each and every item in your data source. This is perfectly fine if all you want is a single bullet in front of the link, which is surrounded by other content.
However, since I didn't know your entire code, I assumed you wanted a <li> for each item in the data source and use the[list] element as a container. Normally, this is accomplished like this:
Code:
<asp:Repeater ...>
<HeaderTemplate>[list]</HeaderTemplate>
<FooterTemplate></ul></FooterTemplate>
<ItemTemplate><li>...</li></ItemTemplate>
</asp:Repeater>
Here the Footer and Header define the outer[list] element. The ItemTemplate than defines individual <li> elements.
But, in your case, it seems to make sense, as you have other content in the ItemTemplate as well.
Cheers,
Imar
---------------------------------------
Imar Spaanjaars
http://Imar.Spaanjaars.Com
Everyone is unique, except for me.
Author of
Beginning ASP.NET 3.5 : in C# and VB,
ASP.NET 2.0 Instant Results and
Dreamweaver MX 2004
Want to be my colleague? Then check out this post.