Open samvmdev opened 2 years ago
Name | Link |
---|---|
Latest commit | 54a5f710510eaf4684159264c6e1e2ddf6c3438f |
Latest deploy log | https://app.netlify.com/sites/amazing-borg-b90f51/deploys/63135bc818907a0008360d69 |
Deploy Preview | https://deploy-preview-51--amazing-borg-b90f51.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
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 Reshama, yes. Here is a preview. I just updated the branch.
Hi @greenhacks (cc: iliadeleon , not sure what the Figma design is.)
REF: https://blog.codeforscience.org/announcing-the-newest-event-fund-grantees/
Can we have it in this format, attached view as an example:
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 Figma Screenshot:
Hi @greenhacks I think there are two places where we want to add the reading time.
I think, for both, we can do as follows:
Author dd mmm yy * x min read
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