Closed Bug 891604 Opened 9 years ago Closed 9 years ago

[ga-IE] Search engine setup for Firefox Mobile for Irish

Categories

(Mozilla Localizations :: ga-IE / Irish, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
mozilla25

People

(Reporter: gueroJeff, Assigned: kscanne)

References

()

Details

(Keywords: productization)

Attachments

(2 files, 1 obsolete file)

We want to ship Firefox Mobile with a good list of search engines for Irish.

We're going to work towards this in three phases. First, the localization team and the l10n drivers will look at the market for the language and come up with a good set of search providers. Kevin, we'll need your input on this, the guidelines for making recommendations are on <https://wiki.mozilla.org/L10n:Mobile/Productization>.

For en-US, we currently use this set, to which we can fall back in the case of lacking localized alternatives:

> Google (mobile version), Amazon, Twitter, Wikipedia (slightly different to desktop, 32x32 icon)

Once we decide to use a particular search engine (and we test it on a mobile device), l10n drivers will contact the owner and ask for permission, and for specifics like proper search flags. Kevin, we'll likely need your help to at least get the contact information.

For search engines that we already ship with on desktop and that you would like to include in mobile as well, the permission is not needed. Consider using a mobile version of the engine if one is available. Also, please make yourself familiar with instructions regarding the <Description/> element at <https://wiki.mozilla.org/L10n:Integrated_Search_Instructions>.

Once we have a specification on what the search engines for Irish should look like, Milos and the localization team will work on an implementation on that. Milos will create and attach patches for the desired changes and get them reviewed. After a successful review, Milos will land them. Kevin, if you're interested, you could create the patches, too. You'd ask Milos for review on those before landing.

Please don't commit any modifications in browser/searchplugins to the hg repository without a patch reviewed by Milos, Stas, Axel or someone else appointed by the l10n drivers. The data here is sensitive to users and our search partners, thus the extra care here.

The Irish team and Milos will be driving this bug to completion. 

We are aiming for inclusion in Firefox 25, which will become available in Aurora on August 6, 2013.
I currently have Google, Amazon UK, Twitter, Wikipedia (Irish), and focal.ie on aurora.   All of these are used on the desktop as well.   Focal.ie is a useful Irish language dictionary/terminology database; would it be ok to use these five?
(In reply to Kevin P. Scannell from comment #1)
> I currently have Google, Amazon UK, Twitter, Wikipedia (Irish), and focal.ie
> on aurora.   All of these are used on the desktop as well.   Focal.ie is a
> useful Irish language dictionary/terminology database; would it be ok to use
> these five?

Cool. Would you be willing to create a patch with these in it for Fennec and assign it to Milos for review please? https://developer.mozilla.org/en-US/docs/Productization_guide
I think I may already be set up correctly, no patch needed.  How does this look?

http://hg.mozilla.org/releases/l10n/mozilla-aurora/ga-IE/file/90d98b82fdc0/mobile/searchplugins


The only change I'm considering is that focal.ie recently released a mobile version of the site - I can change the plugin to reflect that.
(In reply to Kevin P. Scannell from comment #3)
> I think I may already be set up correctly, no patch needed.  How does this
> look?
> 
> http://hg.mozilla.org/releases/l10n/mozilla-aurora/ga-IE/file/90d98b82fdc0/
> mobile/searchplugins
> 
> 
> The only change I'm considering is that focal.ie recently released a mobile
> version of the site - I can change the plugin to reflect that.

That's a good question. I know that the preference is for desktop sites for Fennec on tablets and mobile sites for Fennec on smartphones, but I'm not sure how the negotiation for that works. Axel, Milos, any info?
Yeah, for tablets we probably want the desktop site, and the mobile site for phones. It'd be good to know if focal.ie does feature detection there and sends you to the right site depending on screen size. That's what wikipedia does.

If not, we can do something similar to bug 850984, and encode different urls depending on tablet or not.
It appears they don't detect screen size - no redirect with the mobile devices I've tried here.
No idea if this works - I just mimicked the baidu.com plugin from bug 850984.
(In reply to Kevin Scannell from comment #7)
> Created attachment 773691 [details] [diff] [review]
> Use m.focal.ie if possible, desktop site otherwise
> 
> No idea if this works - I just mimicked the baidu.com plugin from bug 850984.

Hey Kevin,

Thanks for the patch. Could you review flag flod on that attachment please?

Thanks,
Jeff
Attachment #773691 - Flags: review?(francesco.lodolo)
Comment on attachment 773691 [details] [diff] [review]
Use m.focal.ie if possible, desktop site otherwise

Review of attachment 773691 [details] [diff] [review]:
-----------------------------------------------------------------

The image is not showing on Android because of bug 892924 (the data URI most not contain %).

From a quick test I'd say it's working as expected: full website on a tablet (Nexus 7), mobile website on phone (Galaxy S II), but I'd like to review the complete patch and ask pike for a feedback.
Attachment #773691 - Flags: review?(francesco.lodolo) → review-
Do I need to just wait for a fix for bug 892924 then?  Or can you re-review the same patch when that's fixed?
Attached file images.html
Sorry, I gave too much information for granted.

You can fix this by using a data URI without % (i.e. use the data from the second image in this file).
Thanks flod, I understand now.  Here's the new patch.
Attachment #773691 - Attachment is obsolete: true
Attachment #792351 - Flags: review?(francesco.lodolo)
Comment on attachment 792351 [details] [diff] [review]
Improved patch for using m.focal.ie, correct image

Patch looks good to me: image shows up now, tablet and phone work as expected.

The resulting searchplugin can be tested from here
http://flod.org/testsp

Please wait a bit more before landing this, requesting feedback to pike since I'm pretty new at this.

One concern: none of your searchplugins has a license header, I think it would be appropriate to add one. See for example en-US.
http://hg.mozilla.org/mozilla-central/file/bb025b6949e8/mobile/locales/en-US/searchplugins

Let's what Axel thinks about this too.
Attachment #792351 - Flags: review?(francesco.lodolo)
Attachment #792351 - Flags: review+
Attachment #792351 - Flags: feedback?(l10n)
Comment on attachment 792351 [details] [diff] [review]
Improved patch for using m.focal.ie, correct image

Review of attachment 792351 [details] [diff] [review]:
-----------------------------------------------------------------

Yeah, looks good. I'd move the licensing out to another bug, as none of the xml files for search have a license header.
Attachment #792351 - Flags: feedback?(l10n) → feedback+
Thanks Pike. 

Kevin, in the commit message remember to add a reference to this bug and r=flod, f=pike.

I've filed bug 907348 for the license headers.
Status: NEW → ASSIGNED
Landed on aurora:

http://hg.mozilla.org/releases/l10n/mozilla-aurora/ga-IE/rev/3cbd3de9f6a9

I'll mark as resolved+fixed but let me know if there's anything else I need to do.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Test it on Aurora as soon as it's available and mark the bug as verified ;-)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.