Closed Bug 950204 Opened 10 years ago Closed 10 years ago

Bing search suggestions on desktop do not include locale

Categories

(Firefox :: Search, defect)

x86_64
Windows 8
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 29

People

(Reporter: mconnor, Assigned: mconnor)

References

Details

(Whiteboard: [good first verify])

Attachments

(1 file)

We only use one Bing plugin for all locales, so we don't have the right results in autocomplete for a user's locale.

Compare:

http://api.bing.com/osjson.aspx?query=automobile&form=OSDJAS
http://api.bing.com/osjson.aspx?query=automobile&form=OSDJAS&market=ja-JP
http://api.bing.com/osjson.aspx?query=automobile&form=OSDJAS&market=de-DE
http://api.bing.com/osjson.aspx?query=automobile&form=OSDJAS&market=en-US

The obvious fix is to add market={moz:locale} to the bing plugin as a param to suggestions, waiting on Bing to verify that's the correct thing to do.
Verified in Fx26 DE that we're getting en-US suggestions in .de, which is rather suboptimal.  This is the correct param, per Bing.
Attachment #8349740 - Flags: review?(gavin.sharp)
Comment on attachment 8349740 [details] [diff] [review]
bingDesktopLangFix

Exacerbates bug 670451, but that's more than made up for by the UX win.
Attachment #8349740 - Flags: review?(gavin.sharp) → review+
Note, this will send "market=de" instead of "market=de-DE". I didn't see a difference in the sample query, just stating this in case we need to make that explicit to the bing folks.
https://hg.mozilla.org/integration/mozilla-inbound/rev/8e64c79bb855

Axel, that's a good point, I'll flag it for the Bing folks.
https://hg.mozilla.org/mozilla-central/rev/8e64c79bb855
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 29
(In reply to Axel Hecht [:Pike] from comment #4)
> Note, this will send "market=de" instead of "market=de-DE". I didn't see a
> difference in the sample query, just stating this in case we need to make
> that explicit to the bing folks.

It does make a difference for me in Sweden. "market=de" does not give me the same results as "market=de-DE". Instead I get the same results as "market=sv-SE" (and "market=sv") gives me. It looks like Bing do not recognize short locale codes and use other means to guess the users locale, like their IP address.
Whiteboard: [good first verify]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: