Bug tracker

This ticket has been moved to our new tracker. Open Ticket PHPBB3-8836 now.

Minor cosmetic issue - hyphen vs. dash (fix completed in vcs)

This is a minor cosmetic issue, certainly not a show-stopper.

It occurred to me while looking at the new 3.0.6 feeds, that the statistics line would look better using the more typographically correct em dash (— / '—' / "\xE2\x80\x94"), rather than hyphen/minus ('-' / "\x2D"), since we're using UTF-8.

Here's what we have in 3.0.6-RC4:
Statistics: Posted by Murph - Sat, 31 Oct 2009 01:12:48 GMT - Replies 0 - Views 8

Here's what I'm suggesting:
Statistics: Posted by Murph — Sat, 31 Oct 2009 01:12:48 GMT — Replies 0 — Views 8

There are 4 cases of this in feed.php, in adjust_item, at the end of each class, similar to:
Code: Select all
$item_row['statistics'] = $user->lang['POSTED'] . ' ' . $user->lang['POST_BY_AUTHOR'] . ' ' . $user_link . ' - ' . $user->format_date($time). ' - ' . $user->lang['REPLIES'] . ' ' . $row['topic_replies'] . ' - ' . $user->lang['VIEWS'] . ' ' . $row['topic_views'];


There are also a few places elsewhere which could similarly be changed to the UTF-8 dash.

In memberlist.php:
Code: Select all
'VISITED'               => (empty($last_visit)) ? ' — ' : $user->format_date($last_visit),


In viewforum.php:
Code: Select all
page_header($user->lang['VIEW_FORUM'] . ' — ' . $forum_data['forum_name'], true, $forum_id);


In viewtopic.php:
Code: Select all
page_header($user->lang['VIEW_TOPIC'] . ' — ' . $topic_data['topic_title'], true, $forum_id);


In styles/prosilver/template/forumlist_body.html:
Code: Select all
<!-- IF S_ENABLE_FEEDS and forumrow.S_FEED_ENABLED --><!-- <a class="feed-icon-forum" title="{L_FEED} - {forumrow.FORUM_NAME}" href="{U_FEED}?f={forumrow.FORUM_ID}"><img src="{T_THEME_PATH}/images/feed.gif" alt="{L_FEED} - {forumrow.FORUM_NAME}" /></a> --><!-- ENDIF -->


In styles/prosilver/template/overall_header.html, titles in 6 link lines:
Code: Select all
<link rel="alternate" type="application/atom+xml" title="{L_FEED} - {SITENAME}" href="{U_FEED}" />


There's almost certainly a few other cases lurking around, but I think that's the most obvious ones on the main board pages.

Comments / History

Edited ticket

Action performed by Paul.J.Murphy (QA Team) on Nov 3rd 2009, 11:13

Assigned ticket to user "bantu"

Action performed by bantu (3.0 Release Manager) on Nov 24th 2009, 00:04

Changed ticket status from "New" to "Reviewed"

Action performed by bantu (3.0 Release Manager) on Nov 24th 2009, 00:04

Changed ticket status from "Reviewed" to "Fix in progress"

Action performed by bantu (3.0 Release Manager) on Nov 24th 2009, 14:52

Linked ticket with changeset: r10285

Action performed by bantu (3.0 Release Manager) on Nov 24th 2009, 16:00

Changed ticket status from "Fix in progress" to "Fix completed in SVN"

Action performed by bantu (3.0 Release Manager) on Dec 11th 2009, 15:22

Ticket details

Related SVN changesets