-
Notifications
You must be signed in to change notification settings - Fork 10
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 #21
CSS tweaks #21
Conversation
This also fixes #16 |
Looks pretty good to me! On the CoFest page I last tried bumping the headings up to H1/H2 from what they had been before (H3/H4 probably), so with this new CSS they could go back down if they are too big relative to the paragraph text size. |
That looks much better--thank you!! We'll need to check all the pages after you make the change, because some layout stuff, picture sizes, etc. may need adjusting. |
I agree, based on the screenshots this is a worthy improvement. The risk assessment seems fine too - let’s merge it? |
The current |
Documenting deployment logged as #23. |
okay - I think the CSS tweakage will be an ongoing work for now. This PR addresses:
Plenty more needs to be done but I think this is a respectable chunk. The changes are relatively low-risk and should be safe to deploy to prod I think (famous last words, I know) - they're largely font sizes, with some padding and margins, and one html class change. HUGE thanks to @peterjc and @heuermh for the visual feedback to get here.
Example collaborationfest page, before:
after:
