Improve pdf icon style

sagemath · Nov 13, 2023 · 3cb86b5 · 3cb86b5
1 parent 6b39c68
commit 3cb86b5
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 13 deletions.
diff --git a/src/doc/common/static/custom-furo.css b/src/doc/common/static/custom-furo.css
@@ -11,3 +11,13 @@ body[data-theme="dark"] div.highlight {
   background: #383838;
 }
 
+/* Copied the same style for a.pdf from website/templates/index_furo.html */
+
+a.pdf {
+  margin-right: 0.2em;
+}
+
+a.pdf:hover {
+  text-decoration: none;
+}
+
diff --git a/src/doc/en/website/templates/index_furo.html b/src/doc/en/website/templates/index_furo.html
@@ -10,10 +10,13 @@
     border: none;
   }
   a.pdf {
-    margin-left: 0.5em;
-  {%- if hide_pdf_links %}
+    margin-right: 0.2em;
+    {%- if hide_pdf_links %}
     display: none;
-  {%- endif %}
+    {%- endif %}
+  }
+  a.pdf:hover {
+    text-decoration: none;
   }
   table.contentstable {
     align: center;
@@ -41,7 +44,7 @@ <h2>
    <tr valign="top">
     <td width="50%">
       <p class="biglink">
-        <a title="Download PDF" class="pdf" href="../../pdf/en/a_tour_of_sage/a_tour_of_sage.pdf">
+        <a title="PDF" class="pdf" href="../../pdf/en/a_tour_of_sage/a_tour_of_sage.pdf">
           <img class="icon" src="_static/pdf.png"></img>
         </a>
         <a class="biglink" href="a_tour_of_sage/index.html">
@@ -55,7 +58,7 @@ <h2>
     </td>
     <td width="50%">
       <p class="biglink">
-        <a title="Download PDF" class="pdf" href="../../pdf/en/prep/prep_tutorials.pdf">
+        <a title="PDF" class="pdf" href="../../pdf/en/prep/prep_tutorials.pdf">
           <img class="icon" src="_static/pdf.png"></img>
         </a>
         <a class="biglink" href="prep/index.html">
@@ -75,7 +78,7 @@ <h2>
   <tr valign="top">
     <td width="50%">
       <p class="biglink">
-        <a title="Download PDF" class="pdf" href="../../pdf/en/tutorial/sage_tutorial.pdf">
+        <a title="PDF" class="pdf" href="../../pdf/en/tutorial/sage_tutorial.pdf">
           <img class="icon" src="_static/pdf.png"></img>
         </a>
         <a class="biglink" href="tutorial/index.html">
@@ -90,7 +93,7 @@ <h2>
     </td>
     <td width="50%">
       <p class="biglink">
-        <a title="Download PDF" class="pdf" href="../../pdf/en/constructions/constructions.pdf">
+        <a title="PDF" class="pdf" href="../../pdf/en/constructions/constructions.pdf">
           <img class="icon" src="_static/pdf.png"></img>
         </a>
         <a class="biglink" href="constructions/index.html">
@@ -108,7 +111,7 @@ <h2>
   <tr valign="top">
     <td width="50%">
       <p class="biglink">
-        <a title="Download PDF" class="pdf" href="../../pdf/en/thematic_tutorials/thematic_tutorials.pdf">
+        <a title="PDF" class="pdf" href="../../pdf/en/thematic_tutorials/thematic_tutorials.pdf">
           <img class="icon" src="_static/pdf.png"></img>
         </a>
         <a class="biglink" href="thematic_tutorials/index.html">
@@ -128,7 +131,7 @@ <h2>
     </td>
     <td width="50%">
       <p class="biglink">
-        <a title="Download PDF" class="pdf" href="../../pdf/en/faq/faq.pdf">
+        <a title="PDF" class="pdf" href="../../pdf/en/faq/faq.pdf">
           <img class="icon" src="_static/pdf.png"></img>
         </a>
         <a class="biglink" href="faq/index.html">
@@ -184,7 +187,7 @@ <h2>
   <tr valign="top">
     <td width="50%">
       <p class="biglink">
-        <a title="Download PDF" class="pdf" href="../../pdf/en/installation/installation.pdf">
+        <a title="PDF" class="pdf" href="../../pdf/en/installation/installation.pdf">
           <img class="icon" src="_static/pdf.png"></img>
         </a>
         <a class="biglink" href="installation/index.html">
@@ -201,7 +204,7 @@ <h2>
     </td>
     <td width="50%">
       <p class="biglink">
-        <a title="Download PDF" class="pdf" href="../../pdf/en/developer/developer.pdf">
+        <a title="PDF" class="pdf" href="../../pdf/en/developer/developer.pdf">
           <img class="icon" src="_static/pdf.png"></img>
         </a>
         <a class="biglink" href="developer/index.html">

diff --git a/src/sage_docbuild/builders.py b/src/sage_docbuild/builders.py
@@ -720,14 +720,14 @@ def pdf(self):
         #
         # Change the third form to
         #
-        #   <a href="module/module.pdf">blah <img src="_static/pdf.png" /></a>
+        #   <a href="module/module.pdf"><img src="_static/pdf.png">blah</a>
         #
         rst = re.sub(r'`([^`\n]*)`__.*\n\n__ (.*)',
                      r'<a href="\2">\1</a>.', rst)
         rst = re.sub(r'`([^<\n]*)\s+<(.*)>`_',
                      r'<a href="\2">\1</a>', rst)
         rst = re.sub(r':doc:`([^<]*?)\s+<(.*)/index>`',
-                     r'<a href="../../../pdf/en/reference/\2/\2.pdf"><img src="_static/pdf.png"/></a>&nbsp;<a href="\2/index.html">\1</a> ', rst)
+                     r'<a title="PDF" class="pdf" href="../../../pdf/en/reference/\2/\2.pdf"><img src="_static/pdf.png"></a><a href="\2/index.html">\1</a> ', rst)
         # Body: add paragraph <p> markup.
         start = rst.rfind('*\n') + 1
         end = rst.find('\nUser Interfaces')