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

remove typography html #61

Closed
zroupas opened this issue Sep 2, 2019 · 3 comments
Closed

remove typography html #61

zroupas opened this issue Sep 2, 2019 · 3 comments

Comments

@zroupas
Copy link

zroupas commented Sep 2, 2019

Hi and what a great theme to be able to use and customize!

I'm trying to remove Typography from navigation but cannot do that without breaking the page in github. I'm sure there is something I'm missing but it would be great if someone could help.

In a local jekyll setup i can see the _site folder but not in the repo that is uploaded and handled in github.

Thanks

@bencentra
Copy link
Owner

I just pushed some other changes to the theme. With the latest code you can either delete the page, or remove the main_nav: true from the page's front matter. The links in the header and footer are auto-generated and should update accordingly.

@zroupas
Copy link
Author

zroupas commented Sep 3, 2019

Thanks for the help! Are there specific commits that i can checkout files from and add them to my repo?
Or maybe another way that will not affect my changes i'm not aware of and sorry the inconvenience.

I can see that the main_nav changes are in

  • about.md
  • _includes/nav_links.html
  • posts.html
  • typography.html

so if i checkout those and remove main_nav:true from the typography it would one of the solutions suggested?

Regards and thanks for taking the time to update your work for me

@zroupas
Copy link
Author

zroupas commented Sep 6, 2019

Hi again,

For anyone that maybe stumble upon this, you need

_includes/footer.html
_includes/header.html
_includes/nav_links.html

and to add main_nav: true to about.md and posts.html.

Thanks for your time Ben and effort.

@zroupas zroupas closed this as completed Sep 6, 2019
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

No branches or pull requests

2 participants