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

Load dogwood theme as submodule #18

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

jacobwhall
Copy link

This PR moves all (most?) dogwood theme code into a submodule of the dogwood repository itself, in the /dogwood directory. This allows for newer versions of the theme to be applied to this site by updating the submodule. I made a couple of further changes to restore this site to what it looked like before this PR, for example I changed color defaults to their original values using the _sass/dogwood/custom-variables.scss file. See my commit notes below for more information.

My branch successfully builds this PR, hopefully it will continue to work once merged.

@nicolelaine, it was nice to meet you at SOTMUS!

- these will be loaded using a submodule in future
  commits
- to be honest, I copied this verbatim from my
  MapRVA/maprva.github.io repo (of the same license),
  perhaps it could be streamlined
- I didn't see a mono logo available in this repo for
  Missing Maps, so I set the regular one to be the mono
  one too in _config.yml so that the default dogwood one
  wouldn't display.
- this prevents the dogwood theme from displaying the
  icon in the first place
- also, bump version of configure-pages Action
@jacobwhall
Copy link
Author

Also, nota bene that local development will require explicitly cloning the submodule. This can be done with git clone --recurse-submodules ...

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