939752 - Update opensearch plugin

Status: VERIFIED FIXED

(Participation Infrastructure :: Phonebook, defect)

Description

[Giorgos Logiotatidis [:giorgos]]

Mozillians has opensearch support but the plugin need icon update and code cleanup. Let's do these and advertise it!

Comment 1

[[github robot]]

Commits pushed to master at https://github.com/mozilla/mozillians

https://github.com/mozilla/mozillians/commit/472388abd0e679b1b3b5051f44fd42643e34e398
[fix bug 939752] Update opensearch plugin.

- Update favicon
- Cleanup search syntax
- Use absolutify helper.

https://github.com/mozilla/mozillians/commit/ca9cf1fb70c9836779ecf8834ce697015e65c112
Merge pull request #723 from glogiotatidis/939752

[fix bug 939752] Update opensearch plugin.

Comment 2

[Giorgos Logiotatidis [:giorgos]]

I mistakenly removed the text/html Url

Comment 3

[[github robot]]

Commits pushed to master at https://github.com/mozilla/mozillians

https://github.com/mozilla/mozillians/commit/e4a2de15c61a4dbff4d76a2a2777a5b212b674d5
[fix bug 939752] Add Url with text/html.

https://github.com/mozilla/mozillians/commit/b9f9784a4c283494f66bd8725b81fca88b75f713
Merge pull request #737 from glogiotatidis/opensearch-url

[fix bug 939752] Add Url with text/html.

Comment 4

[Rebecca Billings [:rbillings]]

Please add verification steps or mark as [qa-], thanks

Comment 5

[Justin Crawford [:hoosteeno] [:jcrawford]]

I verified on stage. Steps to verify:

1) Visit mozillians.allizom.org
2) Using the search dropdown, look for "Add Mozillians" and click it
3) Verify that you can now choose Mozillians as the default search provider

Comment 6

[Rebecca Billings [:rbillings]]

Added it to FF search providers, icon displayed, search matches mozillians search.

Comment 7

[Andrei [:sancus]]

On prod, this is returning the old dino head for the favicon instead of the (m). The culprit is the data uri ("<Image>data:image/png</Image>" etc) in http://www.mozillians.org/en-US/opensearch.xml -- no matter what I do, I can't seem to get it to refresh and return the (m) instead of the dino head. Hopefully this is just some excessively aggressive caching. If it doesn't clear up by tomorrow, we can reopen and look into it further.