Drupal
/
drupal
mirror of https://git.drupalcode.org/project/drupal.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

- Patch #45228 and patch #45025 by m3avrck and Uwe: aggregator module HTML/CSS cleanups.

4.7.x
Dries Buytaert 2006-01-20 09:09:18 +00:00
parent 8c02d4ec93
commit 9c4d4fe7b2
6 changed files with 63 additions and 74 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
misc/drupal.css
View File

@ -192,8 +192,15 @@ tr.odd .form-item, tr.even .form-item {
/* /*
** Module specific styles ** Module specific styles
*/ */
#aggregator .feed img { #aggregator .feed-source .feed-title {
margin-top: 0;
}
#aggregator .feed-source .feed-image img {
margin-bottom: 0.75em;
}
#aggregator .feed-source .feed-icon {
float: right; float: right;
display: block;
} }
#aggregator .news-item { #aggregator .news-item {
clear: both; clear: both;
@ -206,7 +213,7 @@ tr.odd .form-item, tr.even .form-item {
margin-top: 1em; margin-top: 1em;
margin-left: 1em; margin-left: 1em;
} }
#aggregator .news-item .body .feed { #aggregator .news-item .body .feed-source {
font-size: 0.9em; font-size: 0.9em;
} }
#aggregator .news-item .title { #aggregator .news-item .title {
@ -234,15 +241,15 @@ tr.odd .form-item, tr.even .form-item {
.book .title { .book .title {
font-weight: bold; font-weight: bold;
font-size: 1.3em; font-size: 1.3em;
margin-bottom:1em; margin-bottom: 1em;
} }
.book .tree { .book .tree {
padding-top:1em; padding-top: 1em;
border-top: 1px solid #888; border-top: 1px solid #888;
padding-bottom: 1em; padding-bottom: 1em;
} }
.book .name { .book .name {
padding-top:1em; padding-top: 1em;
font-weight: bold; font-weight: bold;
font-size: 1em; font-size: 1em;
} }

57
modules/aggregator.module
View File

@ -423,7 +423,8 @@ function aggregator_refresh($feed) {
} }
if ($image['LINK'] && $image['URL'] && $image['TITLE']) { if ($image['LINK'] && $image['URL'] && $image['TITLE']) {
$image = '<a href="'. $image['LINK'] .'"><img src="'. $image['URL'] .'" alt="'. $image['TITLE'] .'" /></a>'; // Note, we should really use theme_image() here but that only works with local images it won't work with images fetched with a URL unless PHP version > 5
$image = '<a href="'. check_url($image['LINK']) .'" class="feed-image"><img src="'. check_url($image['URL']) .'" alt="'. check_plain($image['TITLE']) .'" /></a>';
} }
else { else {
$image = NULL; $image = NULL;
@ -908,7 +909,7 @@ function aggregator_page_source() {
$feed = db_fetch_object(db_query('SELECT * FROM {aggregator_feed} WHERE fid = %d', arg(2))); $feed = db_fetch_object(db_query('SELECT * FROM {aggregator_feed} WHERE fid = %d', arg(2)));
$info = theme('aggregator_feed', $feed); $info = theme('aggregator_feed', $feed);
return _aggregator_page_list('SELECT * FROM {aggregator_item} WHERE fid = '. $feed->fid .' ORDER BY timestamp DESC, iid DESC', arg(3), "<div class=\"feed\">$info</div>"); return _aggregator_page_list('SELECT * FROM {aggregator_item} WHERE fid = '. $feed->fid .' ORDER BY timestamp DESC, iid DESC', arg(3), $info);
} }
/** /**
@ -1148,26 +1149,19 @@ function aggregator_page_categories() {
* @ingroup themeable * @ingroup themeable
*/ */
function theme_aggregator_feed($feed) { function theme_aggregator_feed($feed) {
$output = ''; $output = '<div class="feed-source">';
$output .= theme('feed_icon', $feed->url) ."\n";
$output .= $feed->image . ' <h3 class="feed-title"><a href="'. check_url($feed->link) .'">'. check_plain($feed->title) ."</a></h3>\n";
$output .= '<div class="feed-description"><em>'. t('Description:') .'</em> '. filter_xss($feed->description) ."</div>\n";
if ($feed->image) {
$output .= $feed->image;
}
$output .= $feed->description;
$output .= '<h3>'. t('URL') ."</h3>\n";
$output .= theme('xml_icon', $feed->url);
$output .= '<a href="'. check_url($feed->link) .'">'. check_plain($feed->link) ."</a>\n";
$output .= '<h3>'. t('Last update') ."</h3>\n";
$updated = t('%time ago', array('%time' => format_interval(time() - $feed->checked))); $updated = t('%time ago', array('%time' => format_interval(time() - $feed->checked)));
if (user_access('administer news feeds')) { if (user_access('administer news feeds')) {
$output .= l($updated, 'admin/aggregator'); $updated = l($updated, 'admin/aggregator');
}
else {
$output .= $updated;
} }
$output .= '<div class="feed-updated"><em>'. t('Updated:') . "</em> $updated</div>";
$output .= "</div>\n";
return $output; return $output;
} }
@ -1214,27 +1208,25 @@ function theme_aggregator_summary_item($item) {
* @ingroup themeable * @ingroup themeable
*/ */
function theme_aggregator_page_item($item) { function theme_aggregator_page_item($item) {
static $last;
$date = format_date($item->timestamp, 'custom', 'Ymd');
if ($date != $last) {
$last = $date;
$output .= '<h3>'. format_date($item->timestamp, 'custom', 'F j, Y') ."</h3>\n";
}
$output .= "<div class=\"news-item\">\n";
$output .= ' <div class="date">'. format_date($item->timestamp, 'custom', 'H:i') ."</div>\n";
$output .= " <div class=\"body\">\n";
$source = ''; $source = '';
if ($item->ftitle && $item->fid) { if ($item->ftitle && $item->fid) {
$source = '<span class="source">'. l($item->ftitle, "aggregator/sources/$item->fid") .":</span>\n"; $source = l($item->ftitle, "aggregator/sources/$item->fid", array('class' => 'feed-item-source')) . ' -';
} }
$output .= ' <div class="title">'. $source .'<a href="'. check_url($item->link) .'">'. check_plain($item->title) ."</a></div>\n"; if (date('Ymd', $item->timestamp) == date('Ymd')) {
$source_date = t('%ago ago', array('%ago' => format_interval(time() - $item->timestamp)));
}
else {
$source_date = format_date($item->timestamp, 'custom', variable_get('date_format_medium', 'D, Y-m-d H:i'));
}
$output .= "<div class=\"feed-item\">\n";
$output .= '<h3 class="feed-item-title"><a href="'. check_url($item->link) .'">'. check_plain($item->title) ."</a></h3>\n";
$output .= "<div class=\"feed-item-meta\">$source <span class=\"feed-item-date\">$source_date</span></div>\n";
if ($item->description) { if ($item->description) {
$output .= ' <div class="description">'. $item->description ."</div>\n"; $output .= '<div class="feed-item-body">'. filter_xss($item->description) ."</div>\n";
} }
$result = db_query('SELECT c.title, c.cid FROM {aggregator_category_item} ci LEFT JOIN {aggregator_category} c ON ci.cid = c.cid WHERE ci.iid = %d ORDER BY c.title', $item->iid); $result = db_query('SELECT c.title, c.cid FROM {aggregator_category_item} ci LEFT JOIN {aggregator_category} c ON ci.cid = c.cid WHERE ci.iid = %d ORDER BY c.title', $item->iid);
@ -1243,10 +1235,9 @@ function theme_aggregator_page_item($item) {
$categories[] = l($category->title, 'aggregator/categories/'. $category->cid); $categories[] = l($category->title, 'aggregator/categories/'. $category->cid);
} }
if ($categories) { if ($categories) {
$output .= ' <div class="categories">'. t('Categories') .': '. implode(', ', $categories) ."</div>\n"; $output .= '<div class="feed-item-categories">'. t('Categories') .': '. implode(', ', $categories) ."</div>\n";
} }
$output .= " </div>\n";
$output .= "</div>\n"; $output .= "</div>\n";
return $output; return $output;

57
modules/aggregator/aggregator.module
View File

@ -423,7 +423,8 @@ function aggregator_refresh($feed) {
} }
if ($image['LINK'] && $image['URL'] && $image['TITLE']) { if ($image['LINK'] && $image['URL'] && $image['TITLE']) {
$image = '<a href="'. $image['LINK'] .'"><img src="'. $image['URL'] .'" alt="'. $image['TITLE'] .'" /></a>'; // Note, we should really use theme_image() here but that only works with local images it won't work with images fetched with a URL unless PHP version > 5
$image = '<a href="'. check_url($image['LINK']) .'" class="feed-image"><img src="'. check_url($image['URL']) .'" alt="'. check_plain($image['TITLE']) .'" /></a>';
} }
else { else {
$image = NULL; $image = NULL;
@ -908,7 +909,7 @@ function aggregator_page_source() {
$feed = db_fetch_object(db_query('SELECT * FROM {aggregator_feed} WHERE fid = %d', arg(2))); $feed = db_fetch_object(db_query('SELECT * FROM {aggregator_feed} WHERE fid = %d', arg(2)));
$info = theme('aggregator_feed', $feed); $info = theme('aggregator_feed', $feed);
return _aggregator_page_list('SELECT * FROM {aggregator_item} WHERE fid = '. $feed->fid .' ORDER BY timestamp DESC, iid DESC', arg(3), "<div class=\"feed\">$info</div>"); return _aggregator_page_list('SELECT * FROM {aggregator_item} WHERE fid = '. $feed->fid .' ORDER BY timestamp DESC, iid DESC', arg(3), $info);
} }
/** /**
@ -1148,26 +1149,19 @@ function aggregator_page_categories() {
* @ingroup themeable * @ingroup themeable
*/ */
function theme_aggregator_feed($feed) { function theme_aggregator_feed($feed) {
$output = ''; $output = '<div class="feed-source">';
$output .= theme('feed_icon', $feed->url) ."\n";
$output .= $feed->image . ' <h3 class="feed-title"><a href="'. check_url($feed->link) .'">'. check_plain($feed->title) ."</a></h3>\n";
$output .= '<div class="feed-description"><em>'. t('Description:') .'</em> '. filter_xss($feed->description) ."</div>\n";
if ($feed->image) {
$output .= $feed->image;
}
$output .= $feed->description;
$output .= '<h3>'. t('URL') ."</h3>\n";
$output .= theme('xml_icon', $feed->url);
$output .= '<a href="'. check_url($feed->link) .'">'. check_plain($feed->link) ."</a>\n";
$output .= '<h3>'. t('Last update') ."</h3>\n";
$updated = t('%time ago', array('%time' => format_interval(time() - $feed->checked))); $updated = t('%time ago', array('%time' => format_interval(time() - $feed->checked)));
if (user_access('administer news feeds')) { if (user_access('administer news feeds')) {
$output .= l($updated, 'admin/aggregator'); $updated = l($updated, 'admin/aggregator');
}
else {
$output .= $updated;
} }
$output .= '<div class="feed-updated"><em>'. t('Updated:') . "</em> $updated</div>";
$output .= "</div>\n";
return $output; return $output;
} }
@ -1214,27 +1208,25 @@ function theme_aggregator_summary_item($item) {
* @ingroup themeable * @ingroup themeable
*/ */
function theme_aggregator_page_item($item) { function theme_aggregator_page_item($item) {
static $last;
$date = format_date($item->timestamp, 'custom', 'Ymd');
if ($date != $last) {
$last = $date;
$output .= '<h3>'. format_date($item->timestamp, 'custom', 'F j, Y') ."</h3>\n";
}
$output .= "<div class=\"news-item\">\n";
$output .= ' <div class="date">'. format_date($item->timestamp, 'custom', 'H:i') ."</div>\n";
$output .= " <div class=\"body\">\n";
$source = ''; $source = '';
if ($item->ftitle && $item->fid) { if ($item->ftitle && $item->fid) {
$source = '<span class="source">'. l($item->ftitle, "aggregator/sources/$item->fid") .":</span>\n"; $source = l($item->ftitle, "aggregator/sources/$item->fid", array('class' => 'feed-item-source')) . ' -';
} }
$output .= ' <div class="title">'. $source .'<a href="'. check_url($item->link) .'">'. check_plain($item->title) ."</a></div>\n"; if (date('Ymd', $item->timestamp) == date('Ymd')) {
$source_date = t('%ago ago', array('%ago' => format_interval(time() - $item->timestamp)));
}
else {
$source_date = format_date($item->timestamp, 'custom', variable_get('date_format_medium', 'D, Y-m-d H:i'));
}
$output .= "<div class=\"feed-item\">\n";
$output .= '<h3 class="feed-item-title"><a href="'. check_url($item->link) .'">'. check_plain($item->title) ."</a></h3>\n";
$output .= "<div class=\"feed-item-meta\">$source <span class=\"feed-item-date\">$source_date</span></div>\n";
if ($item->description) { if ($item->description) {
$output .= ' <div class="description">'. $item->description ."</div>\n"; $output .= '<div class="feed-item-body">'. filter_xss($item->description) ."</div>\n";
} }
$result = db_query('SELECT c.title, c.cid FROM {aggregator_category_item} ci LEFT JOIN {aggregator_category} c ON ci.cid = c.cid WHERE ci.iid = %d ORDER BY c.title', $item->iid); $result = db_query('SELECT c.title, c.cid FROM {aggregator_category_item} ci LEFT JOIN {aggregator_category} c ON ci.cid = c.cid WHERE ci.iid = %d ORDER BY c.title', $item->iid);
@ -1243,10 +1235,9 @@ function theme_aggregator_page_item($item) {
$categories[] = l($category->title, 'aggregator/categories/'. $category->cid); $categories[] = l($category->title, 'aggregator/categories/'. $category->cid);
} }
if ($categories) { if ($categories) {
$output .= ' <div class="categories">'. t('Categories') .': '. implode(', ', $categories) ."</div>\n"; $output .= '<div class="feed-item-categories">'. t('Categories') .': '. implode(', ', $categories) ."</div>\n";
} }
$output .= " </div>\n";
$output .= "</div>\n"; $output .= "</div>\n";
return $output; return $output;

2
themes/bluemarine/style.css
View File

@ -272,7 +272,7 @@ table {
/* /*
** Module specific styles ** Module specific styles
*/ */
#aggregator .feed { #aggregator .feed-source {
background-color: #eee; background-color: #eee;
border: 1px solid #ccc; border: 1px solid #ccc;
padding: 1em; padding: 1em;

2
themes/chameleon/common.css
View File

@ -145,7 +145,7 @@ br {
.form-item textarea { .form-item textarea {
font-size: 1em; font-size: 1em;
} }
#aggregator .feed { #aggregator .feed-source {
border: 1px solid gray; border: 1px solid gray;
padding: 1em; padding: 1em;
} }

2
themes/pushbutton/style.css
View File

@ -482,7 +482,7 @@ table#footer-menu {
.content .active { .content .active {
color: #369; color: #369;
} }
#aggregator .feed { #aggregator .feed-source {
background-color: #eee; background-color: #eee;
border: 1px solid #ccc; border: 1px solid #ccc;
padding: 1em; padding: 1em;