Skip to content

Conversation

@sumitshinde-84
Copy link
Collaborator

@sumitshinde-84 sumitshinde-84 commented Jul 18, 2025

Description

Screen.Recording.2025-07-18.at.9.08.34.PM.mov

Related Issue(s)

Checklist

  • I have read the contribution guidelines
  • I have considered the performance impact of these changes
  • Suitable unit/system level tests have been added and they pass
  • Documentation has been updated

@@ -0,0 +1,43 @@
{%- set postsToShow = collections.posts | relatedPosts -%}
Copy link
Member

Choose a reason for hiding this comment

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

Why not have the relatedPosts filter always return 3 posts?

Comment on lines 35 to 42
<style>
.line-clamp-2 {
overflow: hidden;
display: -webkit-box;
-webkit-line-clamp: 2;
-webkit-box-orient: vertical;
}
</style>
Copy link
Member

Choose a reason for hiding this comment

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

Why inline the stylesheets like this? It becomes a maintiance mess very quickly.

@Yndira-E We should not allow this, have one stylesheet

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.

3 participants