The "misc" folder contains a file, "drupal.css," which includes a number of CSS tags that affect the display of content generated by the aggregator module. The following tag creates a line break problem:

#aggregator .news-item .title {
float: left;
}

In order to make sure that the next field ("description") breaks to a new line, another tag has been included, as follows:

#aggregator .news-item .description {
clear: both;
}

The problem is that if there is no description field, the field which follows immediately after the title is "source," and since source doesn't have a "clear: both" attribute, the "source" line runs together with the title. To fix this, add the following tag to drupal. css:

#aggregator .news-item .source {
clear: both;
}

By the way, it's rather frustrating that all of these aggregator-related style tags are buried in the misc folder. The first place where most people are likely to look for style tags will be in their theme directory. The second place they might look would be in the directory for the affected module. CSS tags buried in the misc directory are hard to find and easy to overlook. We spent several hours poring over the style tags in our theme directory before we finally figured out where this bug was located.

CommentFileSizeAuthor
#1 aggregator_css.patch503 bytesUwe Hermann

Comments

Uwe Hermann’s picture

Version: 4.6.0 » x.y.z
Status: Active » Needs review
StatusFileSize
new503 bytes

Here's a patch for HEAD (untested).

drumm’s picture

Still applies (somewhat): Hunk #1 succeeded at 216 with fuzz 2 (offset -27 lines).

I would like to see this looked over (maybe re-rolled) and tested (or marked won't fix).

webernet’s picture

Status: Needs review » Closed (fixed)

Code in question was removed (fixed?) by http://drupal.org/node/45228