This patch adds smaller character counts for teasers: 25, 50, 75 and 100.

25 is approx one line on a three col layout, or these famous fixed blog things. 50 on wider layouts and 75 is onle line on two col fluid layout. Hundred is about two lines on a lot of layouts. hort teasers often look ery good ni listings.

For example http://fixme.remixreading.org/taxonomy/term/31. Note that this is achieved with a hard string trim, we want this to be achieved with the theaser generation code, snice that is much nicer, it looks for spaces etc. Our code does not do that yet, while this patch does.

CommentFileSizeAuthor
node_teaser_count.patch2.67 KBBèr Kessels

Comments

robin monks’s picture

+1!!!

I haven't testest the patch. But we need this functionability!

Robin

Bèr Kessels’s picture

I have one +1 so far. Yet another patch that seems to sink away in the mud. please comment, or otherwise close. We can really not afford maintining small patches like these for ages!

jo1ene’s picture

I've tested this and it works. I say +1 for this. It doesn't hurt anything to implement it and there's no "code overhead" to worry about. I say go with it.

jo1ene’s picture

Status: Needs review » Reviewed & tested by the community

What's to review?

dries’s picture

Status: Reviewed & tested by the community » Needs work

Use drupal_map_assoc() with a custom callback that uses format_plural(). Take a look at _aggregator_items() if you need an example.

Jaza’s picture

Version: x.y.z » 4.7.x-dev
Status: Needs work » Closed (fixed)

Smaller teaser values are currently available in HEAD. Closing issue.