Skip to content

Conversation

@pdobacz
Copy link
Member

@pdobacz pdobacz commented May 14, 2024

This is quite rough and manual (see instructions in HTML comments), but maybe good to kick us off? The styling and htmling is probably horrible, fixing this very welcome.

@pdobacz pdobacz changed the title Add the rediness matrix Add the readiness matrix May 14, 2024
@pdobacz pdobacz requested review from axic and hugo-dc May 14, 2024 18:44
<div class="statistics" style="margin: 0px auto; width: 70%; font-size: 70%">
<!-- **NOTE**: How to obtain the HTML table? -->
<!-- 1. Copy source (including reference links) from https://github.com/ipsilon/eof/blob/main/spec/implementation_matrix.md -->
<!-- 2. Paste into https://parsedown.org/demo, convert to HTML -->
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

looks more or less the same

Copy link
Collaborator

@hugo-dc hugo-dc left a comment

Choose a reason for hiding this comment

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

Changing the font size for td and th to 1.3em makes it a bit bigger and easier to read:

table td {
padding: 0.75em 0.75em;
}
table th {
font-size: 0.9em;
font-weight: 400;
padding: 0 0.75em 0.75em 0.75em;
text-align: left;
}

@pdobacz pdobacz requested review from axic and hugo-dc May 16, 2024 07:42
@pdobacz
Copy link
Member Author

pdobacz commented May 16, 2024

Changing the font size for td and th to 1.3em makes it a bit bigger and easier to read:

I increased the font size a bit, but not as much to have the entire table fit on desktop screen easily. It still might be too large to read comfortably on mobile. I lack html-fu to come up with a better fitting design.

@axic axic merged commit 728afc6 into main May 30, 2024
@pdobacz pdobacz deleted the readinessssss branch May 31, 2024 13:13
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.

4 participants