I'm just wondering if it was possible to add the administrative option to only show the first x characters of a classified ad when the user is browsing. I feel like the default action now can result in a lot of scrolling through items the user is not interested in (including potentially large pictures, if the user has used html for images). I think the ability to truncate or remove the description (making the user click on the title of the ad) would be more streamlined.

Comments

mattbk’s picture

mattbk commented
Status: Active » Closed (duplicate)

Oops, this was addressed here: http://drupal.org/node/161027

No solution was yet given as of this posting.

dsellers154’s picture

dsellers154 commented

seems like this module is a bit un-usable untill there is a teaser function built in, preferrably with just the titile line as the teaser (like Cra@#$%ist). How is this module supposed to be used with full nodes displayed? you have to scroll down a full page per ad... Great module btw, just needs the "must have" features...

Dave

milesgillham’s picture

milesgillham commented
Priority: Normal » Critical
Status: Closed (duplicate) » Active

I quite agree. I'll add to the next release when I get some time later in the week. Note that the 5.x branch won't be fixed or back-ported unless sufficient demand (see #161027: How to trim ads in listing view to display title only

Cheers,

Miles

dsellers154’s picture

dsellers154 commented

Hey Miles, thanks for looking into this. I was thinking of using views, but not sure that is a good fix either... Thanks so much

milesgillham’s picture

milesgillham commented

Views would allow you to do your own blocks and pages, it's true. But it sounds like in this instance dakotaroar is referring to general browsing such as category listings. I propose a solution as follows:

  1. On the settings page a checkbox to indicate whether any list of ads will include the body by default or just the teaser/title. That will actually solve this specific problem but doesn't build in any future flexibility.
  2. Now it's probably possible to retheme output with the present function hooks but I think I'd rather make ed_classified themable through templates. Then, anyone who wants to go further to entirely tailor the ad presentation can do so in the normal "Drupal" way which is by overriding theme templates (xxx.tpl.php)

Any comments on that approach? Otherwise, that's what I'll do. Might take a bit more time as I'm still getting my head around how to do it.

Cheers,

Miles

dsellers154’s picture

dsellers154 commented

A checkbox for the teaser title would be AWESOME!

Dave

milesgillham’s picture

milesgillham commented

The checkbox is added (and tested) with -dev release. An alpha5 release will be out in a couple of days which will incorporate this and some other bug fixes as a bonus.

Note that this doesn't strictly truncate to x chars - it just only displays the title as a teaser rather than the entire body. There are quite a few variations to display, going themable is the only way to truly allow for every permutation.

I'll take the conversion to themable templates out of this request and lodge it as a separate feature request as I think it's a good idea anyway. One step closer to being able to emulate c*****list. :-)

Cheers,

Miles

milesgillham’s picture

milesgillham commented
Status: Active » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.