Closed Bug 827221 Opened 7 years ago Closed 7 years ago

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

Categories

(Mozilla Localizations :: ja / Japanese, defect)

All
Android
defect
Not set

Tracking

(firefox18 affected, firefox19 affected, firefox20 fixed)

RESOLVED FIXED
Tracking Status
firefox18 --- affected
firefox19 --- affected
firefox20 --- fixed

People

(Reporter: tetsuharu, Unassigned)

Details

(Keywords: productization)

Attachments

(1 file, 1 obsolete file)

Attached patch patch (obsolete) — Splinter Review
Keywords: productization
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?
Attachment #698570 - Flags: review?(l10n) → review-
Attached patch patch v2Splinter Review
Attachment #698570 - Attachment is obsolete: true
Attachment #698937 - Flags: review?(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).
Attachment #698937 - Flags: review?(milos)
Attachment #698937 - Flags: review?(l10n)
Attachment #698937 - Flags: review+
Attachment #698937 - Flags: review?(l10n) → review+
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.