-
-
Notifications
You must be signed in to change notification settings - Fork 14
Add reading time to post layout #51
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
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for amazing-borg-b90f51 ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
|
@greenhacks Thanks for this PR. Are we able to reduce the size of the font so it matches the size of "Data Umbrella" author? |
|
Hi @greenhacks REF: https://blog.codeforscience.org/announcing-the-newest-event-fund-grantees/ |
|
Hi @reshamas, I was working off the original Figma design for this PR; I'd be glad to try working on the new design. Figma link: https://www.figma.com/file/rs7dsdmKlxdfuQ0wobp5B2/Data-Umbrella |




This PR will work for individual blog posts. There will need to be some edits made to the preview page (article.html) in order to have the correct reading time to show up, perhaps due to how it is calculated. Currently, reading time shows up with the same time for all previews.
What kind of change does this PR introduce?
This PR adds the reading time indicator to the main post page (e.g. "5 minutes to read")
Does this PR introduce a breaking change?
Unsure
What needs to be documented once your changes are merged?
Perhaps a changelog or some kind of tracker indicating that reading time has been implemented. Otherwise, none.
Snapshot from local host