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

VERIFIED FIXED in mozilla25

Status

Mozilla Localizations
ga-IE / Irish
VERIFIED FIXED
4 years ago
4 years ago

People

(Reporter: gueroJeff, Assigned: Kevin Scannell)

Tracking

({productization})

unspecified
mozilla25
x86
Mac OS X
productization

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

4 years ago
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.
(Assignee)

Comment 1

4 years ago
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?
(Reporter)

Comment 2

4 years ago
(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
(Assignee)

Comment 3

4 years ago
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.
(Reporter)

Comment 4

4 years ago
(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.
(Assignee)

Comment 6

4 years ago
It appears they don't detect screen size - no redirect with the mobile devices I've tried here.
(Assignee)

Comment 7

4 years ago
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.
(Reporter)

Comment 8

4 years ago
(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
(Assignee)

Updated

4 years ago
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-
(Assignee)

Comment 10

4 years ago
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?
Created attachment 792191 [details]
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).
(Assignee)

Comment 12

4 years ago
Created attachment 792351 [details] [diff] [review]
Improved patch for using m.focal.ie, correct image

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
(Assignee)

Comment 16

4 years ago
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
Last Resolved: 4 years ago
Resolution: --- → FIXED
Test it on Aurora as soon as it's available and mark the bug as verified ;-)
(Assignee)

Updated

4 years ago
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.