Related Posts: limit number of characters in post titles, instead of number of words

[jeherve]

We currently limit the number of words to 5 in the Related Posts' titles:
https://github.com/Automattic/jetpack/blob/master/modules/related-posts/jetpack-related-posts.php#L686

While that seems to do the trick for the standard 3 posts we display in Jetpack, if blog owners use the jetpack_relatedposts_filter_options filter to display more than 3 posts and have them displayed in a grid, it can create issues:

Should we limit the number of characters instead, and add an ellipsis?

Reported here:

• http://wordpress.org/support/topic/problem-with-related-posts-1?replies=12
• http://wordpress.org/support/topic/related-posts-problem-on-all-phones?replies=1

[jeherve]

Also reported here:
https://wordpress.org/support/topic/serious-issue-with-jetpack?replies=2

[RCowles]

As a workaround, you could add the following to your custom CSS to clear the first post in each row:

.jp-relatedposts-post:nth-child(3n+4) {
    clear: both;
}

[jeherve]

Also reported here:
https://wordpress.org/support/topic/display-issue-with-related-posts-when-showing-6-instead-of-3?replies=3

[xyu]

@jeherve does the CSS fix that @RCowles mentioned work?

[jeherve]

@xyu Yes, it works.

[RCowles]

@xyu Let me know if you'd like me to submit a PR.

[xyu]

@RCowles yeah that would be great.