Skip to content
This repository was archived by the owner on Jan 26, 2025. It is now read-only.

Uncommon sense formatting #244

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Kevin-Kuzniczci
Copy link

These are some format changes that, overall, stay true to the wireframe for Uncommon Sense page. There are some slight differences and there is probably room for improvement on my part, but hopefully these changes are found to be useful!

Just something odd VSCode picked up
The meat and bones of the Uncommon Sense restructuring
Got confused at first and was lost as to which file was connected to which.
@sydva
Copy link
Contributor

sydva commented Feb 9, 2021

Thanks so much for making these changes! However, you might want to read a bit about how Jekyll layouts. The page shouldn't have an HTML tag and whatnot for example--they already use a layout that adds all that stuff in automatically! Sorry this wasn't more clear in our readme.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants