827221 - [fennec][ja] Update twitter search query URI for mobile

Status: RESOLVED FIXED

(Mozilla Localizations :: ja / Japanese, defect)

Description

[Tetsuharu OHZEKI [:tetsuharu] (UTC+9)]

I seem we can update twitter search query URI to "https://mobile.twitter.com/search/"

ref: http://hg.mozilla.org/mozilla-central/file/66d595814554/mobile/locales/en-US/searchplugins/twitter.xml#l8

Current ja: http://code.google.com/p/mozja/source/browse/ja/mobile/searchplugins/twitter-ja.xml?name=l10n-central

Comment 1

[Tetsuharu OHZEKI [:tetsuharu] (UTC+9)]

Attachment: patch

Comment 2

[Axel Hecht [:Pike]]

Comment on attachment 698570 [details] [diff] [review]
patch

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

Thanks for the patch, but we should fix this for both the search and the search form. Mind updating the patch, and requesting review from Milos on the new one?

Comment 3

[Tetsuharu OHZEKI [:tetsuharu] (UTC+9)]

Attachment: patch v2

Comment 4

[Milos Dinic [:Milos]]

Comment on attachment 698937 [details] [diff] [review]
patch v2

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

Even though new default search url doesn't show search window per se, most probably because it doesn't really exist for mobile, this is r+ as it shows latest trands, which is close enough for me.

Please commit with appropriate commit message(ie. mentioning this bug and my review).

Comment 5

[Masahiko Imanaka [:marsf]]

Updated for 20.

http://hg.mozilla.org/releases/l10n/mozilla-aurora/ja/rev/21e410c1a865
http://hg.mozilla.org/releases/l10n/mozilla-aurora/ja-JP-mac/rev/9e165e13ba03

Updating for firefox19 is too late.