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

Added "author" in footer instead of double title #22

Merged
merged 1 commit into from
Aug 29, 2016
Merged

Added "author" in footer instead of double title #22

merged 1 commit into from
Aug 29, 2016

Conversation

lorthirk
Copy link

I wrote this PR because I thought that having site.title shown twice in the footer was redundant, so I introduced the "author" variable in _config.yml to be shown just above the email address. Moreover, this behavior is optional, since if you don't define the "author" variable it falls back to the "title" variable.

Added linke "author" variable in _config.yml
@benbalter
Copy link
Contributor

LGTM.

Thanks @lorthirk!

@benbalter benbalter merged commit 48657bc into jekyll:master Aug 29, 2016
benbalter added a commit that referenced this pull request Aug 29, 2016
@lorthirk lorthirk deleted the author-in-footer branch September 23, 2016 15:46
@jekyll jekyll locked and limited conversation to collaborators Apr 26, 2019
antoniotre86 pushed a commit to antoniotre86/antoniotre86.github.io-old that referenced this pull request Sep 10, 2021
Fix bug causing non-HTML pages to be displayed in tiles.

Closes jekyll#22

See merge request andrewbanchich/forty-jekyll-theme!6
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants