-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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 blog styling #40
Conversation
</ul> | ||
|
||
<article> | ||
{{#with collections.blog.[0]}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So this will display the entire first blog post …
What's the status of this? I'd like to get improved styling merged. |
The blog posts seemed long, so I put a blurb in and the Read the full post link. Would the maintainers rather have the full post, or the blurb and read more? I can finish the implementation of either. |
That last commit was just a merge with the latest master |
@mikeal Do you want to show an excerpt of the latest post (see screenshot)? I'm not entirely happy with the result at the moment, but that's a matter of the post format right now. Should we add a short (1-2 sentences) "tl;dr" for the weekly updates, which is only shown on the blog index page? |
Is this addressed now by #154? |
@fhemberger Yes, it is. A very positive response on that issue, so I think that one will be preferred. +1 for close. |
* chore:Added social icon LinkedIn #40 * Update components/__design__/social-logos.stories.tsx Signed-off-by: Brian Muenzenmeyer <brian.muenzenmeyer@gmail.com> --------- Signed-off-by: Brian Muenzenmeyer <brian.muenzenmeyer@gmail.com> Co-authored-by: sridevi <sridevi.sn> Co-authored-by: Brian Muenzenmeyer <brian.muenzenmeyer@gmail.com>
Added styling for the blog including a blurb at the top for the current blog post and styling the previous posts