FIX: Remove timezone in brackets from user card (#9567)

For clarity and to save space remove the timezone in brackets e.g. (EDT) from the user card. Also add a title to the user time span to say it is Local Time.
This commit is contained in:
Martin Brennan 2020-04-29 08:45:38 +10:00 committed by GitHub
parent 0d519c78fb
commit 17ca47af1a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 11 additions and 12 deletions

View File

@ -77,7 +77,7 @@ export default Component.extend(CardContentsBase, CanCheckEmails, CleansUp, {
@discourseComputed("userTimezone")
formattedUserLocalTime(timezone) {
return moment.tz(timezone).format(I18n.t("dates.time_with_zone"));
return moment.tz(timezone).format(I18n.t("dates.time"));
},
@discourseComputed("username")

View File

@ -171,7 +171,7 @@
</span>
{{/if}}
{{#if showUserLocalTime}}
<span class="local-time">
<span class="local-time" title="{{i18n "local_time"}}">
{{d-icon "far-clock"}}
<span>{{this.formattedUserLocalTime}}</span>
</span>

View File

@ -1444,6 +1444,7 @@ en:
mute: Mute
unmute: Unmute
last_post: Posted
local_time: "Local Time"
time_read: Read
time_read_recently: "%{time_read} recently"
time_read_tooltip: "%{time_read} total time read"

View File

@ -25,11 +25,10 @@ QUnit.test("user card local time", async assert => {
assert.ok(invisible(".user-card"), "user card is invisible by default");
await click("a[data-user-card=eviltrout]:first");
let expectedTime =
moment
.tz("Australia/Brisbane")
.add(-2, "hours")
.format("hh:mm a") + " (AWST)";
let expectedTime = moment
.tz("Australia/Brisbane")
.add(-2, "hours")
.format("h:mm a");
assert.ok(visible(".user-card"), "card should appear");
assert.equal(
@ -51,11 +50,10 @@ QUnit.test("user card local time", async assert => {
await click("a[data-user-card=charlie]:first");
expectedTime =
moment
.tz("Australia/Brisbane")
.add(-14, "hours")
.format("hh:mm a") + " (EDT)";
expectedTime = moment
.tz("Australia/Brisbane")
.add(-14, "hours")
.format("h:mm a");
assert.equal(
find(".user-card .local-time")