docs: relax CSS context match for pretty tables

We currently only render pretty tables if they have the "top_table"
class set. All of our tables set this, except for the ACL & migration
doc tables, which should have set it, and the API reference which does
not want it.

Simplify life by rendering all tables in a pretty style and remove the
need for the "top_table" class entirely. A small rule turns off the
pretty style for the API reference where tables are a hack used to
render enums with horizontal alignment.

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>

diff --git a/docs/aclpolkit.html.in b/docs/aclpolkit.html.in
index 68e6d399b2a85e2794a759313b2c58d36446de52..4a8877d5e7e948c2ecc3014d5c7f914038d9e740 100644 (file)
--- a/docs/aclpolkit.html.in
+++ b/docs/aclpolkit.html.in
@@ -64,7 +64,7 @@
     </p>
 
     <h3><a id="object_connect">virConnectPtr</a></h3>
-    <table class="acl">
+    <table>
       <thead>
         <tr>
           <th>Attribute</th>
@@ -80,7 +80,7 @@
     </table>
 
     <h3><a id="object_domain">virDomainPtr</a></h3>
-    <table class="acl">
+    <table>
       <thead>
         <tr>
           <th>Attribute</th>
@@ -104,7 +104,7 @@
     </table>
 
     <h3><a id="object_interface">virInterfacePtr</a></h3>
-    <table class="acl">
+    <table>
       <thead>
         <tr>
           <th>Attribute</th>
@@ -128,7 +128,7 @@
     </table>
 
     <h3><a id="object_network">virNetworkPtr</a></h3>
-    <table class="acl">
+    <table>
       <thead>
         <tr>
           <th>Attribute</th>
@@ -152,7 +152,7 @@
     </table>
 
     <h3><a id="object_node_device">virNodeDevicePtr</a></h3>
-    <table class="acl">
+    <table>
       <thead>
         <tr>
           <th>Attribute</th>
@@ -172,7 +172,7 @@
     </table>
 
     <h3><a id="object_nwfilter">virNWFilterPtr</a></h3>
-    <table class="acl">
+    <table>
       <thead>
         <tr>
           <th>Attribute</th>
@@ -196,7 +196,7 @@
     </table>
 
     <h3><a id="object_secret">virSecretPtr</a></h3>
-    <table class="acl">
+    <table>
       <thead>
         <tr>
           <th>Attribute</th>
@@ -232,7 +232,7 @@
     </table>
 
     <h3><a id="object_storage_pool">virStoragePoolPtr</a></h3>
-    <table class="acl">
+    <table>
       <thead>
         <tr>
           <th>Attribute</th>
@@ -256,7 +256,7 @@
     </table>
 
     <h3><a id="object_storage_vol">virStorageVolPtr</a></h3>
-    <table class="acl">
+    <table>
       <thead>
         <tr>
           <th>Attribute</th>
@@ -317,7 +317,7 @@
     </p>
 
     <h3><a id="object_connect_driver">Connection Driver Name</a></h3>
-    <table class="acl">
+    <table>
       <thead>
         <tr>
           <th>Connection Driver</th>

diff --git a/docs/genaclperms.pl b/docs/genaclperms.pl
index e20b4c11c389ccc051fceeec1026111aa8ca7ffc..0de2cfad4d53163c41cdecc772b8f055a5311cec 100755 (executable)
--- a/docs/genaclperms.pl
+++ b/docs/genaclperms.pl
@@ -85,7 +85,7 @@ foreach my $object (sort { $a cmp $b } keys %perms) {
     my $olink = lc "object_" . $object;
     print <<EOF;
 <h3><a id="$olink">$class</a></h3>
-<table class="acl">
+<table>
   <thead>
     <tr>
       <th>Permission</th>

diff --git a/docs/libvirt.css b/docs/libvirt.css
index 399404ca5402a34f749ee67464f7d1bde0a3ae7d..d2e1842b6241c200519ad68f02c68d1b76f3cd3a 100644 (file)
--- a/docs/libvirt.css
+++ b/docs/libvirt.css
@@ -161,37 +161,37 @@ p.image {
     text-align: center;
 }
 
-.top_table {
+table {
     border-collapse: collapse;
     min-width: 60%;
     margin-left: auto;
     margin-right: auto;
 }
 
-.top_table th {
+table th {
     background: rgb(0, 95, 97);
     color: rgb(255, 255, 255);
     padding: 0.5em;
 }
 
-.top_table th a {
+table th a {
     color: inherit;
     text-decoration: inherit;
 }
 
-.top_table td, .top_table th {
+table td, table th {
     border: 1px solid rgb(60, 133, 124);
 }
 
-.top_table td {
+table td {
     padding: 4px;
 }
 
-.top_table tr:hover td, .top_table col:hover td {
+table tr:hover td, table col:hover td {
     background: #eeeeee;
 }
 
-.top_table tr td:hover {
+table tr td:hover {
     background: #c5dbd8;
 }
 
@@ -289,42 +289,12 @@ img.diagram {
     margin-right: auto;
 }
 
-table.data th, table.data td {
-    padding: 0.3em;
-}
-
-table.data {
-    border-spacing: 0px;
-}
-
-table.data thead th {
-    background: rgb(178,178,178);
-    text-align: center;
-}
-
-table.data {
-    border: 1px solid black;
-    border-collapse: collapse;
-}
-
-table.data thead tr th {
-    border: 1px solid black;
-}
-
-table.data tr.head th {
-    border-left: 1px solid black;
-    border-right: 1px solid black;
-}
-
-table.data tbody td {
-    background: rgb(240,240,240);
-}
 
-table.data tbody td.y {
+table tbody td.y {
     background: rgb(220,255,220);
     text-align: center;
 }
-table.data tbody td.n {
+table tbody td.n {
     background: rgb(255,220,220);
     text-align: center;
 }
@@ -377,6 +347,18 @@ table.data tbody td.n {
     text-decoration: none;
 }
 
+.api table td,.api table th {
+    border: 0px;
+}
+
+.api table tr:hover td, .api table col:hover td {
+    background: inherit;
+}
+
+.api table tr td:hover {
+    background: inherit;
+}
+
 dl.variablelist > dt {
     display: block;
     float: left;
@@ -392,21 +374,6 @@ dl.variablelist > dt:after {
     content: ": ";
 }
 
-table.acl {
-    margin: 1em;
-    border-spacing: 0px;
-    border: 1px solid #ccc;
-}
-
-table.acl tr, table.acl td {
-    padding: 0.3em;
-    border: 1px solid #ccc;
-}
-
-table.acl thead {
-    background: #ddd;
-}
-
 div.description pre.code {
     border: 1px dashed grey;
     background-color: inherit;

diff --git a/docs/migration.html.in b/docs/migration.html.in
index 7c345b65b7a9076ccd05660f7168bee829d782d3..355f0e89af983b6505989749e83b9dda50f1e7eb 100644 (file)
--- a/docs/migration.html.in
+++ b/docs/migration.html.in
@@ -257,7 +257,7 @@
       combinations.
     </p>
 
-    <table class="data">
+    <table>
       <thead>
         <tr class="head">
           <th colspan="3">Before migration</th>

diff --git a/docs/newapi.xsl b/docs/newapi.xsl
index 670879dc4871dbfbc4228af6b2ed2483c9df3599..0dc4f7ae52212171df113fb1ff215e910bc7230c 100644 (file)
--- a/docs/newapi.xsl
+++ b/docs/newapi.xsl
@@ -43,7 +43,7 @@
 
     <xsl:if test="count(exsl:node-set($acls)/api[@name=$api]/check) > 0">
       <h5>Access control parameter checks</h5>
-      <table class="acl">
+      <table>
         <thead>
           <tr>
             <th>Object</th>
@@ -56,7 +56,7 @@
     </xsl:if>
     <xsl:if test="count(exsl:node-set($acls)/api[@name=$api]/filter) > 0">
       <h5>Access control return value filters</h5>
-      <table class="acl">
+      <table>
         <thead>
           <tr>
             <th>Object</th>