diff mbox

[dpdk-dev,1/3] doc: set alignments in NIC overview table

Message ID 1487625998-8040-2-git-send-email-thomas.monjalon@6wind.com (mailing list archive)
State Accepted, archived
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Thomas Monjalon Feb. 20, 2017, 9:26 p.m. UTC 
  Some CSS alignments were not explicitly set.
The pointer is also set to default for the table.

Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
---
 doc/guides/nics/overview.rst | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

Comments

John McNamara Feb. 23, 2017, 12:47 p.m. UTC | #1 
> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Thomas Monjalon
> Sent: Monday, February 20, 2017 9:27 PM
> To: dev@dpdk.org
> Subject: [dpdk-dev] [PATCH 1/3] doc: set alignments in NIC overview table
> 
> Some CSS alignments were not explicitly set.
> The pointer is also set to default for the table.
> 
> Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>

Nice.

Acked-by: John McNamara <john.mcnamara@intel.com>
John McNamara Feb. 23, 2017, 12:51 p.m. UTC | #2 
> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Thomas Monjalon
> Sent: Monday, February 20, 2017 9:27 PM
> To: dev@dpdk.org
> Subject: [dpdk-dev] [PATCH 1/3] doc: set alignments in NIC overview table
> 
> Some CSS alignments were not explicitly set.
> The pointer is also set to default for the table.
> 

On a related note, a while ago we looked at rotating the text of the NICs
to make the table slightly more readable.

I've attached an example.

It might be nice to add this as well, if it doesn't interfere with the other
formatting.

John
Thomas Monjalon Feb. 24, 2017, 1:16 p.m. UTC | #3 
2017-02-23 12:51, Mcnamara, John:
> On a related note, a while ago we looked at rotating the text of the NICs
> to make the table slightly more readable.

Me too I've looked at it while ago.

> I've attached an example.
> 
> It might be nice to add this as well, if it doesn't interfere with the other
> formatting.

If I remember well, the issue is that it cannot be applied in PDF version which
will be rendered with large columns outside of the page (driver names on one
line instead of the current "one letter per line").
diff mbox

Patch

diff --git a/doc/guides/nics/overview.rst b/doc/guides/nics/overview.rst
index 2c7f5eb..3b733f7 100644
--- a/doc/guides/nics/overview.rst
+++ b/doc/guides/nics/overview.rst
@@ -53,10 +53,15 @@  Most of these differences are summarized below.
 .. raw:: html
 
    <style>
+      table#id1 {
+         cursor: default;
+      }
+      table#id1 th, table#id1 td {
+         text-align: center;
+      }
       table#id1 th {
          font-size: 80%;
          white-space: pre-wrap;
-         text-align: center;
          vertical-align: top;
          padding: 2px;
       }
@@ -69,6 +74,7 @@  Most of these differences are summarized below.
       }
       table#id1 td:first-child {
          padding-left: 1em;
+         text-align: left;
       }
    </style>