930563 - [B2G][L10n][Clock] Chinese Traditional: Highlight date instead of first number displayed

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::Clock, defect)

Description

[Peter Pin-Guang Chen [:petercpg] (MozTW.org)]

Attachment: screenshot

The date in en-US is displayed in "Thursday, October 24", while it is "10 月 24 日 星期四 in zh-TW". 
Only the date should be highlighted instead of the first number displayed.

Build: 20131023225434
Gaia: c57951d06f6c9fa5a762d151e4c75214b988679f
Platform Version: 26.0a2

Comment 1

[Peter Pin-Guang Chen [:petercpg] (MozTW.org)]

Still affects zh-TW locale in 1.3.


Device: Geeksphone Keon
Build: 20140205010019
Gaia: 3405205
Platform Version: 28.0

Comment 2

[Gnaneswar]

Attachment: Date_Screenshot.jpg

Attached screenshot. Please let me know if this what you are expecting.

Comment 3

[Peter Pin-Guang Chen [:petercpg] (MozTW.org)]

(In reply to Gnaneswar from comment #2)
> Created attachment 8371396 [details]
> Date_Screenshot.jpg
> 
> Attached screenshot. Please let me know if this what you are expecting.

No, it doesn't follow locale's date format, "%b %e 日 %A" per string dateFormat. 
You can see that there're numerals for month(10) and day(24), but month was highlighted in the original screenshot.

Comment 4

[Gnaneswar]

In clock_view.js file, in updateDayDate() function, it seems below code snippet highlights the first number.

>> this.dayDate.innerHTML = formated.replace(/([0-9]+)/, '<b>$1</b>');

Comment 5

[Gnaneswar]

I am working on this and will submit patch soon

Comment 6

[Gnaneswar]

Attachment: Patch which highlights date for chinese tradition locale too.

Attached patch which apply bold for date in Chinese tradition locale.

Today,somehow I am not able to push the patch to github.
After entering git hub credentials, there is no progress even after some hours.
Hence, I could not share github PR.

Comment 7

[Peter Pin-Guang Chen [:petercpg] (MozTW.org)]

Comment on attachment 8373304 [details] [diff] [review]
Patch which highlights date for chinese tradition locale too.

Review of attachment 8373304 [details] [diff] [review]:
-----------------------------------------------------------------

This patch fixes the bug on my Keon, but I might not be the right reviewer. Probably someone from Gaia team should give r+?

Mike, I just find you on the Modules page, could you find an appropriate person to make review? Thanks.

Comment 8

[Gnaneswar]

Attachment: Pointer to Github PR

Finally succeeded in pushing the patch. Attached  github PR.

Comment 9

[Marcus Cavanaugh [:mcav]]

Comment on attachment 8373304 [details] [diff] [review]
Patch which highlights date for chinese tradition locale too.

I'm going to steal the review from Mike. Gnaneswar, you're on the right track, but it would be better for us to have a more generic solution. We can do this by mucking about with the localeFormat string before we actually parse out the date.

I'll upload a patch in a second that explains what I mean; I'll have you review it and see if that makes sense to you.

Comment 10

[Marcus Cavanaugh [:mcav]]

Attachment: Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/16186

Gnaneswar, this patch should account for any locale, no matter what format they use for their date. Let me know if this looks good to you, or feel free to change it further if you think it needs something more. Sorry to keep stealing patches from you!

Comment 11

[Gnaneswar]

(In reply to Marcus Cavanaugh [:mcav] <[e-mail]> from comment #10)
> Created attachment 8374408 [details] [review]
> Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/16186

This patch looks good to me.

>Sorry to keep stealing patches from you!
Its fine if it improves the quality of code!

Comment 12

[Marcus Cavanaugh [:mcav]]

Thanks! Landed in https://github.com/mozilla-b2g/gaia/commit/adfc2a2f75e57c4dc6e38df8ff21ca1cd8bb9075.

Comment 13

[Edward Chen[:edchen]]

Verified info:
Gaia      6e71ab4da1b08586ea0c758edb7aa199ee34cd2f
Gecko     https://hg.mozilla.org/mozilla-central/rev/bb030d47c946
BuildID   20140219160202
Version   30.0a1

The bug has fixed, I mark it to VERIFIED.