Remove line-height override CSS #27

Closedrenfredxh opened this issue 4 years ago
renfredxh commented 4 years ago

Setting the line-height to initial makes spacing content elements within threads inconsistent with the rest of sourcegraph. Remove this global style and set it only on the element that need it (list-entry-header etc.)

Write Preview Markdown
dmitshur commented 4 years ago

Go for it (as we discussed), but it's likely that "only the element that need it" = almost all elements except a few that display user content/comment body.

Write Preview Markdown
dmitshur commented 3 years ago

I want to refactor and improve CSS and frontend rendering in general for this project down the line. This specific task isn't really relevant anymore, closing since it's unhelpful to track it individually.

Write Preview Markdown
dmitshur closed this 3 years ago
to comment.