603980 - (fx40-p12n-it) [it] Firefox 4.0 productization review for Italian

Status: RESOLVED FIXED

(Mozilla Localizations :: it / Italian, defect)

Description

[Staś Małolepszy :stas]

As you might have seen in the newsgroups or on the Mozilla blog, there's a couple of changes to search engines planned for the 4.0 release for en-US: <http://groups.google.com/group/mozilla.dev.l10n/msg/81073d8d67e1c4c3>. We would like to use this opportunity to review the current productization setup for Italian and make any changes that you think will be good for our users.

What is a productization review?
================================

Here's what we're interested in:
* search engines,
* protocol handlers,
* feed readers,
* live bookmark.
You can see what your locale currently ships with at <http://hg.mozilla.org/l10n/l10n-src-verification/file/default/reference/browser/2.0/it>

Please take a moment and review the choice of services available by default in your locale. If there have been any new services that have recently gained popularity and that offer good value for the users, please suggest adding them to your locale. On the other hand, if there are services that lost credibility (e.g. display too many adds) or have poor user experience, consider suggesting that we remove them. If in doubt, feel free to make a suggestion and we'll look at it together. You might find the guidelines at <https://wiki.mozilla.org/L10n:Productization> and <https://wiki.mozilla.org/L10n:Firefox/Productization> helpful, too.

Required changes
================

There's two things I would like to draw your attention to (if applicable):

1) if you have a google.xml file in your `searchplugins` directory, please remove it,

2) Bloglines is closing down on November 1st and we need to remove it from the feed reader options for all locales (see bug 595485). We suspect that it will make sense to move Google Reader up to the first position (if your locale has it). We will be offering technical help with this change, either in form of ready-made patches or a mass-landing by one of the l10n-drivers. If you don't think that moving Google Reader up is a good idea for your locale, please let us know!

Changes in en-US and en-GB
==========================

Here's the changes that we will be making in en-US and en-GB:
* add Bing as the 3rd general search engine (see bug 603298),
* remove Answers,
* remove Creative Commons.

We're making these changes in en-US and en-GB because we feel that they follow the recent market trends and user preferences in the US and the UK. We would like to go through the same thought process for your locale and see if any of these changes would make sense in Italian builds as well. If you decide that implementing these changes would be in your users' best interest then please go ahead and include them in the patch you'll attach to this bug.


Implementation
==============

Please make suggestions for changes and attach patches that Seth, Axel or I will review. Please note that since Bing provides a global plugin which does the locale detection itself, you will only need to add "bing" to your list.txt if you decide to include it. The XML file will be automatically pulled from en-US.

Once a review is granted, you can land the patch on l10n-central if your locale actively maintains it. Otherwise, please wait for further instructions which will be put in the comments of this bug and announced in mozilla.dev.l10n (we're still unsure when the branching will happen). Please do not land any changes except for the required ones (google.xml, Bloglines) on 1.9.2.

If you do not wish to make any changes, let us know, and we'll only work on removing Bloglines.

Thanks!

Comment 1

[Francesco Lodolo [:flod]]

I want to ask other opinions before submitting a patch for l10n-central, these are my personal considerations.

No changes needed on Live Bookmarks and Protocol Handlers.

I agree with the decision to move Google Reader as first feed reader.

About searchplugins: 
* I didn't see any new site or service becoming popular in the last years, so I would maintain the existing plug-ins. 
* I'd definitely like to add Bing as a generic search engine
* If we have to keep the total number to 6, we can drop creative commons (I don't think it's much used)

Comment 2

[Francesco Lodolo [:flod]]

Attachment: Patch for mozilla-1.9.2

Since we were already using 3 services, is it possible to add My Yahoo! (like en-US)?

Previous list: Bloglines, Google, Netvibes.
New list (patch): Google, My Yahoo!, Netvibes.

Comment 3

[Seth Bindernagel]

Comment on attachment 482909 [details] [diff] [review]
Patch for mozilla-1.9.2

Reverting to en-US is fine.  Please land this change and annotate the check in with a comment referencing this bug number.

Comment 4

[Staś Małolepszy :stas]

Comment on attachment 482909 [details] [diff] [review]
Patch for mozilla-1.9.2

Thanks flod. What Seth said :)

Comment 5

[Francesco Lodolo [:flod]]

Landed patch on mozilla-1.9.2
http://hg.mozilla.org/releases/l10n-mozilla-1.9.2/it/rev/c27bfa93c42d

About l10n-central: 
* no changes on Live Bookmarks and Protocol Handlers
* same changes for Feed reader options applied to mozilla-1.9.2
* searchplugins: add Bing as a third generic search engine, since it can provide a good value, remove Creative Commons which is not really useful

Comment 6

[Francesco Lodolo [:flod]]

Attachment: Patch l10n-central

New list of feed readers: Google, My Yahoo!, Netvibes.

Searchplugins: removed Creative Commons, added Bing, dispose items in alphabetical order like en-US

Comment 7

[Francesco Lodolo [:flod]]

(In reply to comment #5)
> Landed patch on mozilla-1.9.2
> http://hg.mozilla.org/releases/l10n-mozilla-1.9.2/it/rev/c27bfa93c42d

Verified with Mozilla/5.0 (Windows; U; Windows NT 6.1; it; rv:1.9.2.12pre) Gecko/20101014 Namoroka/3.6.12pre

Comment 8

[Seth Bindernagel]

Comment on attachment 483388 [details] [diff] [review]
Patch l10n-central

This looks good to me.  Please check it into l10n-central if you haven't already.  Please cite bug 603980 in your checkin comment.

Comment 9

[Francesco Lodolo [:flod]]

(In reply to comment #8)
> This looks good to me.  Please check it into l10n-central if you haven't
> already.  Please cite bug 603980 in your checkin comment.

Done
http://hg.mozilla.org/l10n-central/it/rev/8a847db3cf16

Comment 10

[Francesco Lodolo [:flod]]

(In reply to comment #9)
> Done
> http://hg.mozilla.org/l10n-central/it/rev/8a847db3cf16

Still can't verify since this push completely busted trunk nightly builds (bing.xml is missing).

Example: http://tinderbox.mozilla.org/showlog.cgi?log=Mozilla-l10n-it/1287300227.1287300295.28733.gz

Comment 11

[Francesco Lodolo [:flod]]

Attachment: Patch to fix "My Yahoo!" name

Giacomo (prometeo) made me notice that the name "My Yahoo!" it's localized as "Mio Yahoo!". This patch apply to both mozilla-1.9.2 and l10n-central.

Comment 12

[Francesco Lodolo [:flod]]

Temporarily removed Bing from default searchplugins, since nothing seems to move in Bug 603298 and we're still without nightly builds. 
http://hg.mozilla.org/l10n-central/it/rev/ceb5ab28e7a5

I'll add Bing back as soon as bing.xml is available.

Comment 13

[Seth Bindernagel]

Comment on attachment 483921 [details] [diff] [review]
Patch to fix "My Yahoo!" name

Looks good.  Please check in and provide a remark that mentions bug 603980 and this fix.

Comment 14

[Francesco Lodolo [:flod]]

"My Yahoo!" fix verified on mozilla-1.9.2 (Gecko/20101023 Namoroka/3.6.12pre)

Added back bing to trunk, waiting for the next build to verify both changes.

Comment 15

[Francesco Lodolo [:flod]]

Verified on Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0b8pre) Gecko/20101024 Firefox/4.0b8pre

Comment 16

[Francesco Lodolo [:flod]]

Attachment: Patch for mobile-browser

I've just realized that we need to push the same change to feed readers for mobile-browser on l10n-central. Is it ok to use this bug or should I open a new bug?

Comment 17

[Francesco Lodolo [:flod]]

Another problem on l10n-central: the parameter "browser.search.order.3=Bing" is missing in the Italian region.properties. 
http://hg.mozilla.org/mozilla-central/rev/092596c1faef

Should I add it to the current region.properties or provide a new patch and have it reviewed?

Comment 18

[Seth Bindernagel]

Comment on attachment 486097 [details] [diff] [review]
Patch for mobile-browser

Please check this in with a comment that references bug 603980 and this change.

Comment 19

[Seth Bindernagel]

(In reply to comment #17)
> Another problem on l10n-central: the parameter "browser.search.order.3=Bing" is
> missing in the Italian region.properties. 
> http://hg.mozilla.org/mozilla-central/rev/092596c1faef
> 
> Should I add it to the current region.properties or provide a new patch and
> have it reviewed?

Can you just create a quick patch for review?  It will make the record keeping a bit cleaner, even if it is over-doing it.

Comment 20

[Francesco Lodolo [:flod]]

Attachment: Patch to add browser.search.order for Bing

Comment 21

[Seth Bindernagel]

Comment on attachment 486901 [details] [diff] [review]
Patch to add browser.search.order for Bing

Please land and reference this bug # and the change.

Comment 22

[Francesco Lodolo [:flod]]

Commit on l10n-central
http://hg.mozilla.org/l10n-central/it/rev/4d6f114968b9