Mercurial > public > mercurial-scm > hg
diff tests/test-hgwebdir.t @ 24054:fdf7794be41d
hgweb: replace implicit <tbody> with explicit <thead> where appropriate
Some templates in paper style use <tbody> elements inside <table> to assign a
class to "body" part of that table (in this case, to make rows striped). The
problem is that the <tbody> is preceded by <tr> element, which browsers
understand as an implicit start of table body, so the following exlicit <tbody>
will actually be "nested", which is not valid.
Since that first <tr> contains table headers, wrapping it in <thead> is both
semantically correct and follows the advertised XHTML 1.1 doctype.
| author | Anton Shestakov <engored@ya.ru> |
|---|---|
| date | Fri, 06 Feb 2015 15:52:55 +0800 |
| parents | 7a9cbb315d84 |
| children | 78e8890cfb4b |
line wrap: on
line diff
--- a/tests/test-hgwebdir.t Sat Jan 31 12:54:35 2015 -0500 +++ b/tests/test-hgwebdir.t Fri Feb 06 15:52:55 2015 +0800 @@ -201,6 +201,7 @@ <h2 class="breadcrumb"><a href="/">Mercurial</a> </h2> <table class="bigtable"> + <thead> <tr> <th><a href="?sort=name">Name</a></th> <th><a href="?sort=description">Description</a></th> @@ -209,6 +210,7 @@ <th> </th> <th> </th> </tr> + </thead> <tbody class="stripes2"> <tr> @@ -699,6 +701,7 @@ <h2 class="breadcrumb"><a href="/">Mercurial</a> > <a href="/t">t</a> </h2> <table class="bigtable"> + <thead> <tr> <th><a href="?sort=name">Name</a></th> <th><a href="?sort=description">Description</a></th> @@ -707,6 +710,7 @@ <th> </th> <th> </th> </tr> + </thead> <tbody class="stripes2"> <tr> @@ -1128,6 +1132,7 @@ <h2 class="breadcrumb"><a href="/">Mercurial</a> </h2> <table class="bigtable"> + <thead> <tr> <th><a href="?sort=name">Name</a></th> <th><a href="?sort=description">Description</a></th> @@ -1136,6 +1141,7 @@ <th> </th> <th> </th> </tr> + </thead> <tbody class="stripes2"> </tbody>