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

CSS tweaks #70

Merged
merged 2 commits into from
Nov 27, 2024
Merged

CSS tweaks #70

merged 2 commits into from
Nov 27, 2024

Conversation

pcraig3
Copy link
Collaborator

@pcraig3 pcraig3 commented Nov 27, 2024

Summary

This PR includes 2 small CSS tweaks that make the NOFO look a bit nicer:

  1. More space between multi-line anchors in the application checklist
  2. Add a dash () character for quadruple nested bullets.

1. More space between multi-line anchors in the application checklist

Previously, several multiline anchor tags in a row would just show up one after another with equal spaces between all lines, leading to an unclear separation between individual <a> elements.

Make the line length a bit smaller and add some bottom margin to stacked anchors fixes this.

before after
Unclear where line breaks are in second cell with 4 anchor lines Line breaks are more evident
Screenshot 2024-11-27 at 3 49 13 PM Screenshot 2024-11-27 at 3 48 26 PM

2. Add a dash () character for quadruple nested bullets

It's crazy how many bullets some people need.

Once you went past 3, they would repeat themselves. Now, we have a 4th level for visual differentiation.

before after
Screenshot 2024-11-27 at 3 59 37 PM Screenshot 2024-11-27 at 3 59 07 PM

This will help us better distinguish between discrete anchors when
cells have multiple multi-line anchors.
It's not happened before now, but you'd think it would have.
@pcraig3 pcraig3 merged commit 8750ffb into main Nov 27, 2024
4 checks passed
@pcraig3 pcraig3 deleted the css-tweaks branch November 27, 2024 23:32
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.

1 participant