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

Docs: Unusually large padding after table cell lines in Contributor_Tiers.mdx #8805

Open
2 tasks done
JoshuaKGoldberg opened this issue Mar 30, 2024 · 0 comments · May be fixed by #9041
Open
2 tasks done

Docs: Unusually large padding after table cell lines in Contributor_Tiers.mdx #8805

JoshuaKGoldberg opened this issue Mar 30, 2024 · 0 comments · May be fixed by #9041
Labels
accepting prs Go ahead, send a pull request that resolves this issue documentation Documentation ("docs") that needs adding/updating

Comments

@JoshuaKGoldberg
Copy link
Member

Before You File a Documentation Request Please Confirm You Have Done The Following...

Suggested Changes

https://typescript-eslint.io/maintenance/contributor-tiers includes a few table cells that have multiple lines of content. For example, Entry Requirements > Committer. There are <br />s between each <p> in those cells to make separate their content across lines.

Right now, there's something like twice as much vertical padding after each of those lines compared to what we'd expect:

Screenshot of the Contributor Tiers table with 2-3 line-heights of padding between cell lines in Committer and Maintainer

Proposal: let's tweak the styles for those tables to not have so much vertical padding?

Affected URL(s)

https://typescript-eslint.io/maintenance/contributor-tiers/

@JoshuaKGoldberg JoshuaKGoldberg added triage Waiting for maintainers to take a look documentation Documentation ("docs") that needs adding/updating labels Mar 30, 2024
@JoshuaKGoldberg JoshuaKGoldberg added accepting prs Go ahead, send a pull request that resolves this issue and removed triage Waiting for maintainers to take a look labels Apr 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepting prs Go ahead, send a pull request that resolves this issue documentation Documentation ("docs") that needs adding/updating
Projects
None yet
1 participant