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

Add back styling to lists within table cells (fixes #1141) #1252

merged 1 commit into from Mar 29, 2021


Copy link

This overrides the Datasette reset - see

ul {
list-style: none;
- to add back the default styling of list items displayed within Datasette table cells.

Following this change, the same content as in the original issue looks like this:


This overrides the Datasette reset (see,
to add back the default styling of list items displayed within Datasette
table cells.
@@ -452,6 +452,10 @@ table a:link {
margin-left: -10%;
font-size: 0.8em;
.rows-and-columns td ol,ul {
list-style: initial;
list-style-position: inside;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is needed otherwise the bullet point marker appears in the neighbouring cell to the left, rather than the cell with the bullet point content.

Copy link

codecov bot commented Mar 9, 2021

Codecov Report

Merging #1252 (d22aa32) into main (d0fd833) will decrease coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1252      +/-   ##
- Coverage   91.56%   91.51%   -0.05%     
  Files          34       34              
  Lines        4244     4244              
- Hits         3886     3884       -2     
- Misses        358      360       +2     
Impacted Files Coverage Δ
datasette/ 92.93% <0.00%> (-0.75%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d0fd833...d22aa32. Read the comment docs.

Copy link

simonw commented Mar 29, 2021

Neat fix, thank you!

@simonw simonw merged commit e72397d into simonw:main Mar 29, 2021
simonw added a commit that referenced this pull request Mar 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.

None yet

2 participants