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 borders to tables for clarity #76

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aartaka
Copy link

@aartaka aartaka commented Sep 29, 2024

This is because default HTML tables are hard to read due to unclear/absent cell borders. See https://srfi.schemers.org/srfi-48/srfi-48.html#format ~h and the surrounding cells for an example of such a confusion.

An alternative would be to increase spacing between cells, but this is both eating up screen real estate (especially for me with my x2 scaling), and perpetuating the border clarity problem.

@arthurgleckler
Copy link
Contributor

arthurgleckler commented Sep 29, 2024

I see your point, but I don't want to make such a far-reaching change to the look of tables in all SRFIs. However, I have changed srfi.css to add this:

td, th {
  vertical-align: top;
}

This makes tables like the one you referenced in SRFI 48 much easier to read. I'm surprised that it isn't the default for all tables in HTML.

@aartaka
Copy link
Author

aartaka commented Sep 29, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants