HTML and style cleanup for EmailSummary plugin.

This commit is contained in:
Samantha Doherty 2011-02-18 13:49:29 -05:00
parent 6a1b0e2375
commit e9184dd775

View File

@ -124,13 +124,18 @@ class UserEmailSummaryHandler extends QueueHandler
$out = new XMLStringer(); $out = new XMLStringer();
$out->raw(sprintf(_('<p>Recent updates from %1s for %2s:</p>'), $out->elementStart('div', array('width' => '100%',
'style' => 'background-color: #ffffff; border: 4px solid #4c609a; padding: 10px;'));
$out->elementStart('div', array('style' => 'color: #ffffff; background-color: #4c609a; font-weight: bold; margin-bottom: 10px; padding: 4px;'));
$out->raw(sprintf(_('Recent updates from %1s for %2s:'),
common_config('site', 'name'), common_config('site', 'name'),
$profile->getBestName())); $profile->getBestName()));
$out->elementEnd('div');
$out->elementStart('table', array('width' => '550px',
'style' => 'border: none; border-collapse: collapse;', 'cellpadding' => '6'));
$out->elementStart('table', array('width' => '541px', 'style' => 'border: none'));
while ($notice->fetch()) { while ($notice->fetch()) {
$profile = Profile::staticGet('id', $notice->profile_id); $profile = Profile::staticGet('id', $notice->profile_id);
@ -145,30 +150,30 @@ class UserEmailSummaryHandler extends QueueHandler
$out->elementStart('td', array('width' => AVATAR_STREAM_SIZE, $out->elementStart('td', array('width' => AVATAR_STREAM_SIZE,
'height' => AVATAR_STREAM_SIZE, 'height' => AVATAR_STREAM_SIZE,
'align' => 'left', 'align' => 'left',
'valign' => 'top')); 'valign' => 'top',
'style' => 'border-bottom: 1px dotted #C5CEE3; padding: 10px 6px 10px 6px;'));
$out->element('img', array('src' => ($avatar) ? $out->element('img', array('src' => ($avatar) ?
$avatar->displayUrl() : $avatar->displayUrl() :
Avatar::defaultImage(AVATAR_STREAM_SIZE), Avatar::defaultImage(AVATAR_STREAM_SIZE),
'class' => 'avatar photo',
'width' => AVATAR_STREAM_SIZE, 'width' => AVATAR_STREAM_SIZE,
'height' => AVATAR_STREAM_SIZE, 'height' => AVATAR_STREAM_SIZE,
'alt' => $profile->getBestName())); 'alt' => $profile->getBestName()));
$out->elementEnd('td'); $out->elementEnd('td');
$out->elementStart('td', array('align' => 'left', $out->elementStart('td', array('align' => 'left',
'valign' => 'top')); 'valign' => 'top',
'style' => 'border-bottom: 1px dotted #C5CEE3; padding: 10px 6px 10px 6px;'));
$out->element('a', array('href' => $profile->profileurl), $out->element('a', array('href' => $profile->profileurl),
$profile->nickname); $profile->nickname);
$out->text(' '); $out->text(' ');
$out->raw($notice->rendered); $out->raw($notice->rendered);
$out->element('br'); // yeah, you know it. I just wrote a <br> in the middle of my table layout. $out->elementStart('div', array('style' => 'font-size: 0.8em; padding-top: 4px;'));
$noticeurl = $notice->bestUrl(); $noticeurl = $notice->bestUrl();
// above should always return an URL // above should always return an URL
assert(!empty($noticeurl)); assert(!empty($noticeurl));
$out->elementStart('a', array('rel' => 'bookmark', $out->elementStart('a', array('rel' => 'bookmark',
'class' => 'timestamp',
'href' => $noticeurl)); 'href' => $noticeurl));
$dt = common_date_iso8601($notice->created); $dt = common_date_iso8601($notice->created);
$out->element('abbr', array('class' => 'published', $out->element('abbr', array('style' => 'border-bottom: none;',
'title' => $dt), 'title' => $dt),
common_date_string($notice->created)); common_date_string($notice->created));
$out->elementEnd('a'); $out->elementEnd('a');
@ -178,21 +183,23 @@ class UserEmailSummaryHandler extends QueueHandler
if (!empty($convurl)) { if (!empty($convurl)) {
$out->text(' '); $out->text(' ');
$out->element('a', $out->element('a',
array('href' => $convurl.'#notice-'.$notice->id, array('href' => $convurl.'#notice-'.$notice->id),
'class' => 'response'),
_('in context')); _('in context'));
} }
} }
$out->elementEnd('div');
$out->elementEnd('td'); $out->elementEnd('td');
$out->elementEnd('tr'); $out->elementEnd('tr');
} }
$out->elementEnd('table'); $out->elementEnd('table');
$out->raw(sprintf(_('<p><a href="%1s">change your email settings for %2s</a></p>'), $out->raw(sprintf(_('<p><a href="%1s">change your email settings for %2s</a></p>'),
common_local_url('emailsettings'), common_local_url('emailsettings'),
common_config('site', 'name'))); common_config('site', 'name')));
$out->elementEnd('div');
$body = $out->getString(); $body = $out->getString();
// FIXME: do something for people who don't like HTML email // FIXME: do something for people who don't like HTML email