From c0103b2827014045545b878a31993fb456d67d3d Mon Sep 17 00:00:00 2001 From: roman-yagodin Date: Wed, 27 Sep 2017 09:31:10 +0300 Subject: [PATCH] Close #63: Use THEAD tag for table header, also utilize UseAccessibleHeader property to set scope="col" attributes instead of adding them manually --- R7.Documents/ViewDocuments.ascx | 2 +- R7.Documents/ViewDocuments.ascx.cs | 9 +-------- 2 files changed, 2 insertions(+), 9 deletions(-) diff --git a/R7.Documents/ViewDocuments.ascx b/R7.Documents/ViewDocuments.ascx index 10be336..127bc30 100644 --- a/R7.Documents/ViewDocuments.ascx +++ b/R7.Documents/ViewDocuments.ascx @@ -1,7 +1,7 @@ <%@ Control Language="C#" AutoEventWireup="false" CodeBehind="ViewDocuments.ascx.cs" Inherits="R7.Documents.ViewDocuments" %>
+ UseAccessibleHeader="true" OnSorting="grdDocuments_Sorting" OnRowCreated="grdDocuments_RowCreated"> diff --git a/R7.Documents/ViewDocuments.ascx.cs b/R7.Documents/ViewDocuments.ascx.cs index ee67c98..42cea82 100644 --- a/R7.Documents/ViewDocuments.ascx.cs +++ b/R7.Documents/ViewDocuments.ascx.cs @@ -182,15 +182,8 @@ protected void grdDocuments_RowCreated (object sender, GridViewRowEventArgs e) switch (e.Row.RowType) { case DataControlRowType.Header: - // set CSS class for edit column header + e.Row.TableSection = TableRowSection.TableHeader; e.Row.Cells [0].CssClass = "EditHeader"; - - // TODO: Doesn't UseAccessibleHeader=true does same thing? - // setting "scope" to "col" indicates to for text-to-speech - // or braille readers that this row containes headings - for (var count = 1; count <= e.Row.Cells.Count - 1; count++) { - e.Row.Cells [count].Attributes.Add ("scope", "col"); - } break; case DataControlRowType.DataRow: