Closed Bug 606467 Opened 9 years ago Closed 9 years ago

Feedback opens en_US page in localized builds

Categories

(Firefox for Android Graveyard :: General, defect)

x86_64
Linux
defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: tim.babych, Assigned: mfinkle)

Details

Attachments

(1 file)

Tested on today's fennec-4.0b2pre.uk.linux-i686.tar.bz2
What URL would that be? Ad-hoc, can't stab at what to put into an mxr search.
Both urls in http://mxr.mozilla.org/mobile-browser/source/app/profile/extensions/feedback@mobile.mozilla.org/content/overlay.xul#61 should not specify a locale at all.

Locale detection ftw :-)
tracking-fennec: --- → ?
Attached patch patchSplinter Review
Removes the "en-US" from the URLs. Kinda embarrassed I checked code in like that :(
Assignee: nobody → mark.finkle
Attachment #485289 - Flags: review?(21)
(In reply to comment #4)
> Created attachment 485289 [details] [diff] [review]
> patch
> 
> Removes the "en-US" from the URLs. Kinda embarrassed I checked code in like
> that :(

You should be like those buttons : 'sad' because this code has been checked in (and i probably should have seen that when i've moved feedback.dtd :( ), 'happy' because this has been corrected soon :)
pushed:
http://hg.mozilla.org/mobile-browser/rev/c0a7866940d2
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
verified FIXED on builds:
Mozilla/5.0 (Maemo; Linux armv71; rv:2.0b8pre) Gecko/20101025 Namoroka/4.0b8pre Fennec/4.0b2pre

and

Mozilla/5.0 (Android; Linux armv71; rv:2.0b8pre) Gecko/20101025 Namoroka/4.0b8pre Fennec/4.0b2pre
Status: RESOLVED → VERIFIED
tracking-fennec: ? → ---
You need to log in before you can comment on or make changes to this bug.