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

Make sure criterion tables are small but other step 4 tables are large #12

Merged
merged 2 commits into from
Oct 24, 2024

Conversation

pcraig3
Copy link
Collaborator

@pcraig3 pcraig3 commented Oct 24, 2024

Summary

This PR: fixes table widths for step 4, and makes some updates to the README.

We have some general rules for table sizes:

  • tables with 3 or more columns are full-width, OR tables with over 100 words

However, for Step 4, we have more specific ideas:

  • "Criterion" tables should be col-width
  • All other tables should be full-width

This PR fixes our table rules for step 4 to make that work.

before after
only 2 tables here are full-width all tables full-width except our 2 criterion tables
Screenshot 2024-10-24 at 1 59 20 PM Screenshot 2024-10-24 at 1 58 40 PM

Why are we doing this

Technically a bug 🐞

How to test this

Try out CDC 0157 in the NOFO Builder and check out step 4.

We have generic table rules that are being applied, but for step 4
we have specific expectations.

Roughly speaking:

- all 'criterion' tables should be small (normal col width)
- all other tables in step 4 should be full-width
Also update the link to install Poetry (it was pointing to npm).
@pcraig3 pcraig3 assigned pcraig3 and admoorgit and unassigned pcraig3 Oct 24, 2024
@admoorgit admoorgit merged commit 1d1304c into main Oct 24, 2024
1 check passed
@pcraig3 pcraig3 deleted the small-criterion-tables branch October 25, 2024 14:21
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