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

feat(Post): Implement asset cover feature #94

Closed
wants to merge 3 commits into from

Conversation

Venefilyn
Copy link
Contributor

@Venefilyn Venefilyn commented Sep 29, 2019

fixes #31

This adds a few things, assets cover image on posts, post, and home page. With home and posts list it shows the image cropped but centred.

Although not implemented, there's preparation to support landscape type with a new themeConfig coverLayout that defaults to minimal.

Screenshots

Home page

image

Posts list

image

Post detail

image

cc @Quaj @saltysugar


IssueHunt Summary

Referenced issues

This pull request has been submitted to:


IssueHunt has been backed by the following sponsors. Become a sponsor

@egoist
Copy link
Collaborator

egoist commented Sep 30, 2019

doesn't work on the demo

@Venefilyn
Copy link
Contributor Author

Woops, forgot to include demo. See latest change

@Venefilyn
Copy link
Contributor Author

Bump @egoist

<div class="post-title">
{{ post.attributes.title }}
<saber-link class="post post-card" :to="post.attributes.permalink">
<div v-if="post.attributes.assets" class="post-cover">
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

actually post.attributes.assets defaults to {} 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Show post cover image on home/post/posts page
2 participants