Feb 4, 2024 · Feb 2, 2024 · Feb 3, 2024 · Feb 4, 2024 · Feb 4, 2024
diff --git a/python_docs_theme/static/pydoctheme.css b/python_docs_theme/static/pydoctheme.css
    text-decoration: underline 1px;
    text-underline-offset: 0.25em;
 }
 /* No underline for navigation */
 a.headerlink,
 div.genindex-jumpbox a,
 div.modindex-jumpbox a,
 div#search-results a,
 div.sphinxsidebar a,
 div.toctree-wrapper a,
 div[role=navigation] a,
 table.contentstable a,
 table.indextable a {
    text-decoration: none;
 }

 /* Except when hovered */
 div.genindex-jumpbox a:hover,
 div.modindex-jumpbox a:hover,
 div#search-results a:hover,
 div.sphinxsidebar a:hover,
 div.toctree-wrapper a:hover,
 div[role=navigation] a:hover,
 table.contentstable a:hover,
 table.indextable a:hover {
    text-decoration: underline;
    text-underline-offset: auto;
 }

 body {
    margin-left: 1em;
    margin-right: 10px;
 }

 div.footer a {
    text-underline-offset: auto;
 }

 div.footer a:hover {
    color: #0095c4;
 }
Original file line number	Diff line number	Diff line change
Expand Up		@@ -33,6 +33,31 @@ a[href] {
		text-decoration: underline 1px;
		text-underline-offset: 0.25em;
		}
		/* No underline for navigation */
		a.headerlink,
		div.genindex-jumpbox a,
		div.modindex-jumpbox a,
		div#search-results a,
		div.sphinxsidebar a,
		div.toctree-wrapper a,
		div[role=navigation] a,
		table.contentstable a,
		table.indextable a {
		text-decoration: none;
		}

		/* Except when hovered */
		div.genindex-jumpbox a:hover,
		div.modindex-jumpbox a:hover,
		div#search-results a:hover,
		div.sphinxsidebar a:hover,
		div.toctree-wrapper a:hover,
		div[role=navigation] a:hover,
		table.contentstable a:hover,
		table.indextable a:hover {
		text-decoration: underline;
		text-underline-offset: auto;
Copy link Member ezio-melottiFeb 4, 2024 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. `auto` should already be the default -- is there something else overriding this when it's not explicitly stated, or did you add it as a precaution to prevent other rules from overriding the value in case the offset is added to some parent container in the future? Copy link MemberAuthor hugovkFeb 4, 2024 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. is there something else overriding this when it's not explicitly stated ... Yes, it's overridden by this: a[href] {text-decoration: underline1px;text-underline-offset:0.25em;} So "normal" body text links have underlines with extra offset; but navigational links only have underlines for hover, and they're the default offset. ezio-melotti reacted with thumbs up emoji
		}

		body {
		margin-left: 1em;
Expand DownExpand Up		@@ -284,6 +309,10 @@ div.footer {
		margin-right: 10px;
		}

		div.footer a {
		text-underline-offset: auto;
		}

		div.footer a:hover {
		color: #0095c4;
		}
Expand Down