1326030 - [he] Update Morfix searchplugin favicon to its latest

Status: RESOLVED FIXED

(Mozilla Localizations :: he / Hebrew, defect)

Description

[Itiel]

Current favicon of morfix's built-in search provider in latest he builds is not up-to-date.
The updated favicon is:
http://www.morfix.co.il/images/favicon.ico

Comment 1

[Tomer Cohen :tomer]

Mobile Search Plugins:  https://dxr.mozilla.org/l10n-central/source/he/mobile/searchplugins
Mobile Search Plugins list: https://dxr.mozilla.org/l10n-central/source/he/mobile/searchplugins/list.txt

Desktop Search Plugins: https://dxr.mozilla.org/mozilla-central/source/browser/locales/searchplugins
Desktop Search Plugins list: https://dxr.mozilla.org/mozilla-central/rev/dccfc624915b1bebe7975076e9b42bd46684cbc6/browser/locales/search/list.json#303-308

Comment 2

[Francesco Lodolo [:flod]]

While bug 1328079 is still open, we're still shipping this, and we should fix the icon.

Their favicon only includes 32px, and has a heavy drop shadow. I'm using the letter from their flat wordmark to get both 32px and 16px.

Comment 3

[Francesco Lodolo [:flod]]

Attachment: morfix.ico

Icon (.ico with 16+32px)

Comment 4

[Francesco Lodolo [:flod]]

Attachment: Bug 1326030 - [he] Update icon for Morfix searchplugin

Review commit: https://reviewboard.mozilla.org/r/205082/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/205082/

Comment 5

[Mike Kaply [:mkaply]]

Comment on attachment 8934140 [details]
Bug 1326030 - [he] Update icon for Morfix searchplugin

https://reviewboard.mozilla.org/r/205082/#review210622

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/559592144c24
[he] Update icon for Morfix searchplugin r=mkaply

Comment 7

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/559592144c24