-
Notifications
You must be signed in to change notification settings - Fork 3
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
Refactor: Numbered section #136
Conversation
✅ Deploy Preview for cal-itp-website ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
This is now ready for review @angela-tran @thekaveman |
Wait I thought |
@thekaveman Oh my bad. Painful rebase. |
@mrose914 can you please review the content of this numbered section: https://deploy-preview-136--cal-itp-website.netlify.app/ We used the exact wording from the content doc, just want to make sure this is the intended casing (only No. 1 is capitalized, the rest are lower case): |
@thekaveman Could we still merge this w/o waiting on final copy? I can make a new Issue for the copy issue on just the numbered area. This PR moves the giant |
I would suggest we remove “lt” from the first line and leave the four goals to begin with lower case casing. @thekaveman this isn’t a blocking change/approval. We can push to staging and will message to Marty that the copy may undergo non material changes after his sign off. Thank you! |
closes #101
Thanks to https://stackoverflow.com/questions/5732836/html-css-numbered-list-with-numbers-inside-of-circles
What this PR does
Everything between the purple dots,
Bringing industry standards to California’s transit providers
and the purple/pink dots.<ul>
<ol>
and CSS pseudoelement circles