Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #289: Inconsistent table typography sizing #1215

Merged
merged 9 commits into from Aug 26, 2022
23 changes: 23 additions & 0 deletions docs/demo/lists_tables.rst
Expand Up @@ -346,3 +346,26 @@ List Tables

This is a long caption for a figure. Lorem ipsum dolor sit amet, consectetur adipiscing elit.
Donec porttitor dolor in odio posuere, vitae ornare libero mattis. In lobortis justo vestibulum nibh aliquet, non.


Tables with paragraphs
----------------------

.. regression test for https://github.com/readthedocs/sphinx_rtd_theme/issues/289

Test to see that tables behave well with nested paragraphs.

+------------+----------+-------------------------+
| Precedence | Operator | Description |
+============+==========+=========================+
| 1 | \:: | Scope resolution |
+------------+----------+-------------------------+
| 2 | ( ) | Function call |
| | | |
| | [ ] | Subscript |
| | | |
| | . | Member access |
| | | |
| | .{ } | Bit-field concatenation |
+------------+----------+-------------------------+

2 changes: 1 addition & 1 deletion sphinx_rtd_theme/static/css/theme.css

Large diffs are not rendered by default.

7 changes: 6 additions & 1 deletion src/sass/_theme_rst.sass
Expand Up @@ -225,6 +225,12 @@
&:hover .headerlink
opacity: 1

// Consistent font sizing for table typography
.wy-table
th, td
p, ul
font-size: inherit

// override the Wyrm accessibility anti-pattern of hiding button focus
.btn:focus
outline: 2px solid
Expand Down Expand Up @@ -379,7 +385,6 @@
td
border: none
p
font-size: inherit
line-height: inherit
td > strong
display: inline-block
Expand Down