Closed Bug 1643523 Opened 4 years ago Closed 4 years ago

Use different tippytop icons for different Yandex search shortcuts

Categories

(Firefox :: New Tab Page, enhancement, P1)

enhancement

Tracking

()

VERIFIED FIXED
Firefox 79
Tracking Status
firefox79 --- verified

People

(Reporter: nanj, Assigned: nanj)

References

Details

Attachments

(1 file)

This is a follow up for bug 1630764.

Currently, Yandex search shortcut variants share the same url "yandex.com", which points to the same tippytop icon for yandex.com. Per client's request, we should use the Russian edition for yandex.ru.

Priority: -- → P1
Pushed by najiang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e5dc3106541e Use different icons for different Yandex search shortcuts r=mkaply
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 79

STR for QA:

  • Change the pref "browser.search.region" to "RU"
  • Flip the pref "intl.multilingual.enabled", and install the "ru" languange pack from l10n-central
  • Before switching the browser locale to "ru", click on the "Add search engine" from the Topsites context menu. And you should see the Yandex search shortcut for "yandex.com" with the "Y" logo
  • Then switch to the "ru" locale, and restart the browser
  • On the "Add search engine" form, you should see the Yandex logo in Russian
  • Once you add it and click on it, it should take you to quantumbar and search with "yandex.ru"
  • Switch back to "en-US", the pinned Yandex shortcut should also switch back to the logo with "Y"

I have verified that the "Yandex.ru" search shortcut is correctly displayed by using Firefox Nightly 79.0a1 (Build ID: 20200624093107) en-US with the following changes:

  • The value of the "browser.search.region" pref set to "RU".
  • The "ru" language pack installed.
  • The "Language" in the "about:preferences" is set to "Russian".
    I have also verified that switching back to the "English (United States)" language in the "about:preferences" page will correctly display the
    "Yandex.com" search shortcut.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: