CSS Friendly Control Adapters

Re: Does the CSS friendly GridView have problems showing selected rows?

angel eyes — Tue, 24 Oct 2006 06:00:41 GMT

Hey, I'm glad it helped you. Thanks for posting this, too. Sometimes you want to contribute, and don't know if you've helped... So it feels good to know it did.

Re: Does the CSS friendly GridView have problems showing selected rows?

kjarrodc — Mon, 23 Oct 2006 17:19:01 GMT

Thank you for this find. I was able to apply the same fix to my website.

Re: Does the CSS friendly GridView have problems showing selected rows?

Russ Helfand — Fri, 15 Sep 2006 00:57:29 GMT

Good fix. I'll include it in the next rev of the kit. Thanks, this was a valuable contribution.

Re: Does the CSS friendly GridView have problems showing selected rows?

angel eyes — Thu, 14 Sep 2006 20:22:17 GMT

I continued...

The problem originates in the GetRowClass function in GridViewAdapter.vb, the select-case there doesn't allow for the fact that

DataControlRowState is actually a bitwise variable enum.

You see:

DataControlRowState.Alternate = 1

DataControlRowState.Selected = 2

So when an alternate row is selected, the value passed here is 3, which isn't handled by the select-case.

Until MS comes up with a better all-around code, I simply changed one line (number 13 in this post):

1    Private Function GetRowClass(ByVal gridView As GridView, ByVal row As GridViewRow) As String
2     Dim className As String = ""
3    
4     Select Case row.RowState
5        Case DataControlRowState.Alternate
6           className += " AspNet-GridView-Alternate "
7           className += gridView.AlternatingRowStyle.CssClass
8        Case DataControlRowState.Edit
9           className += " AspNet-GridView-Edit "
10          className += gridView.EditRowStyle.CssClass
11       Case DataControlRowState.Insert
12          className += " AspNet-GridView-Insert "
13       Case DataControlRowState.Selected, DataControlRowState.Selected Or DataControlRowState.Alternate
14          className += " AspNet-GridView-Selected "
15          className += gridView.SelectedRowStyle.CssClass
16    End Select
17   
18    Return className
19   End Function

Does the CSS friendly GridView have problems showing selected rows?

angel eyes — Thu, 14 Sep 2006 18:33:38 GMT

It seems it doesn't render the selected rows.

I tried using:

both in the skin file and in the aspx file, and this is after trying to change the CSS marked ".MyGridView .AspNet-GridView table tbody tr.AspNet-GridView-Selected td"

Crawling up the walls, started digging into the code, found out that:

This doesn't occurr for all rows, just for the alternate ones. Normal items are rendered properly.
The reason is simple: the adapter doesn't render a AspNet-GridView-Selected Class for the alternate ones, in fact, it doesn't give ANY class for the selected one and the one before, and the one after.

this is a piece of the rendered HTML

Before: (row number one selected, all is fine)

 <tbody>
	<tr class=" AspNet-GridView-Selected ">
		<td><a href="javascript:__doPostBack('GridView1','Select$0')">Select</a></td>
		<td>Sinh</td>
		<td>17416</td>
		<td>31</td>
		<td>1.7943</td>
	</tr>
	<tr class=" AspNet-GridView-Alternate ">
		<td><a href="javascript:__doPostBack('GridView1','Select$1')">Select</a></td>
		<td>Tanh</td>
		<td>17448</td>
		<td>31</td>
		<td>1.7910</td>
	</tr>
	<tr>
		<td><a href="javascript:__doPostBack('GridView1','Select$2')">Select</a></td>
		<td>Truncate</td>
		<td>18233</td>
		<td>31</td>
		<td>1.7139</td>
	</tr>

And after - row number 2 (alternate item) chosen:

<tbody>
	<tr>
		<td><a href="javascript:__doPostBack('GridView1','Select$0')">Select</a></td>
		<td>Sinh</td>
		<td>17416</td>
		<td>31</td>
		<td>1.7943</td>
	</tr>
	<tr>
		<td><a href="javascript:__doPostBack('GridView1','Select$1')">Select</a></td>
		<td>Tanh</td>
		<td>17448</td>
		<td>31</td>
		<td>1.7910</td>
	</tr>
	<tr>
		<td><a href="javascript:__doPostBack('GridView1','Select$2')">Select</a></td>
		<td>Truncate</td>
		<td>18233</td>
		<td>31</td>
		<td>1.7139</td>
	</tr>