[FFmpeg-devel] [PATCH 1/4] web/style: Only add bottom border when requested to <th>'s in <tbody>

Timothy Gu timothygu99 at gmail.com
Sun Nov 23 07:36:18 CET 2014


When the <th> is a row heading, the border looks as ugly as heck.

Add a special case for <th> in <tbody> as a long heading.

Signed-off-by: Timothy Gu <timothygu99 at gmail.com>
---

This does not have any difference in the website right now, but there will be
in the new fateserver.

---
 src/download        |  2 +-
 src/less/style.less | 17 +++++++++++++++++
 2 files changed, 18 insertions(+), 1 deletion(-)

diff --git a/src/download b/src/download
index 86560fb..d4b433b 100644
--- a/src/download
+++ b/src/download
@@ -213,7 +213,7 @@
     <td>-</td>
     <td><a href="http://fate.ffmpeg.org">fate.ffmpeg.org</a> server software repository</td>
   </tr>
-        <tr>
+        <tr class="table-heading">
           <th colspan="4">Mirrors</th>
         </tr>
   <tr>
diff --git a/src/less/style.less b/src/less/style.less
index 433a78f..83212a4 100644
--- a/src/less/style.less
+++ b/src/less/style.less
@@ -270,11 +270,28 @@ code {
             }
             > th {
                 background-color: @Cmainlightlight;
+            }
+        }
+    }
+    // Table headings
+    > thead {
+        > tr {
+            > th {
                 // Bottom align for column headings
                 border-bottom: 2px solid @Cmainlightlightl;
             }
         }
     }
+    > tbody {
+        > tr.table-heading {
+            // These special rules are useful for long headings in <tbody>
+            // Headings in <thead> or row headings are handled already.
+            > th {
+                border-bottom: 2px solid @Cmainlightlightl;
+                background-color: @Cmainlightlight;
+            }
+        }
+    }
     // Account for multiple tbody instances
     > tbody + tbody {
         border-top: 2px solid @Cmaindarkdark;
-- 
1.9.1



More information about the ffmpeg-devel mailing list