-
-
Notifications
You must be signed in to change notification settings - Fork 74
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
Add continue reading link #42
base: master
Are you sure you want to change the base?
Conversation
@amorphic Thanks for your PR. The buttons are not properly formatted when I tried with attila-demo source code |
update for pelican 4 (arulrajnet#40)
7ca0e46
to
ecd2d12
Compare
Thanks for checking @arulrajnet. The buttons render correctly on my blog so I assumed they were good to go. I'll looking into what's going wrong with the attila demo blog and update the PR accordingly. |
1 similar comment
Thanks for checking @arulrajnet. The buttons render correctly on my blog so I assumed they were good to go. I'll looking into what's going wrong with the attila demo blog and update the PR accordingly. |
Allows a reader to click directly through to the article from the end of the summary.
ecd2d12
to
477a5db
Compare
@arulrajnet I've updated the PR and tested against my blog and the Attila demo site: The change is actually much cleaner since the update too. Please let me know if you want any other changes. |
Are these links optional? I'd like to have the ability to opt out of rendering them. |
This PR:
Adds a "Continue Reading" link at the bottom of each article's summary. This allows the reader to click directly through to the full article from the end of the summary rather than scroll back up to the Headline. The link is styled using the same "button" style as the article tags.
Updates the "Comments" link to the same style as the "Continue Reading" link and places them side-by-side to conserve vertical space: