Closed Bug 603984 (fx40-p12n-kn) Opened 14 years ago Closed 14 years ago

[kn] Firefox 4.0 productization review for Kannada

Categories

(Mozilla Localizations :: kn / Kannada, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: stas, Unassigned)

References

Details

(Keywords: productization, Whiteboard: [l10n-src-verification updated])

Attachments

(2 files, 1 obsolete file)

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 Kannada 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/kn>

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 Kannada 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!
Shankar, please include the addition of mibbit on 1.9.2 in the update to region.properties, too?

I'll reject the current sign-off based on the lack of mibbit.

Best way forward is to just roll bloglines and mibbit into a single patch. The gecko.handlerService.defaultHandlersVersion should be back to 3, too.
The Bloglines change does not affect your locale. 

However, if you have any productization changes to make for Firefox 4, like the Bing suggestion listed in comment 0, please provide a patch to make that change on l10n-central. 

If there are not changes, just close this bug.  Thank you.
Sign-off on http://hg.mozilla.org/l10n-central/kn/pushloghtml?changeset=a3a58b985cc7 rejected, too, please drive the status of your region.properties to a good state here?
Pike,

I just updated the region.properties: 
http://hg.mozilla.org/l10n-central/kn/pushloghtml?changeset=13e1d91d33e2

Does this looks perfect? Can I sign-off now?

Shankar
By the way, why this bug was not listed in webdashboard?

Probably, that was the reason I missing this!

Shankar
I have also added bing to the list.txt

Shankar
Oops sorry!! I should not have landed the changes in l10n-central repository until the patch get approved! Hence reverting the changes done to both region.properties and list.txt back to the previous versions. Will submit the patch for both files in this bug soon.

Sorry for the inconvenience caused!

Shankar
Attached patch patch for region.properties (obsolete) — Splinter Review
Here is the patch file for region.properties

Shankar
Here is the the patch for list.txt
Attachment #492592 - Attachment is patch: true
Attachment #492592 - Attachment mime type: application/octet-stream → text/plain
Comment on attachment 492592 [details] [diff] [review]
patch for region.properties

Putting these into Seth's review queue.

Any reason why you removed netvibes?
Attachment #492592 - Flags: review?(sethb)
Attachment #492593 - Flags: review?(sethb)
There is no particular reason as such. I just copied the content from mozilla-central! :)
Comment on attachment 492592 [details] [diff] [review]
patch for region.properties

I don't think we need to remove Netvibes.  Can you reattach a new patch that doesn't remove Netvibes?  Thanks.
Attachment #492592 - Flags: review?(sethb) → review-
Comment on attachment 492593 [details] [diff] [review]
patch for list.txt

This looks fine to land on l10n-central.  You can consolidate this into the other patch if you wish.  If you land just this patch now, please add a comment that references bug 603984 and this change.
Attachment #492593 - Flags: review?(sethb) → review+
Whiteboard: [update l10n-src-verification]
I have included netvibes in this patch
Attachment #492592 - Attachment is obsolete: true
Comment on attachment 493371 [details] [diff] [review]
patch for region.properties

I cannot view this patch due to the MIME type of octet-stream, it's not viewable.  Can you reattach?
/me edits, hang on
Attachment #493371 - Attachment is patch: true
Attachment #493371 - Attachment mime type: application/octet-stream → text/plain
Comment on attachment 493371 [details] [diff] [review]
patch for region.properties

Here you go, there's an (edit) next to the attachment name in the details view which let's you edit all the glory details on existing attachments.
Thanks, Pike.  Didn't know that.  :)
Comment on attachment 493371 [details] [diff] [review]
patch for region.properties

This looks good.  Please land on l10n-central, adding a comment that references bug 603984 and the change.
Attachment #493371 - Flags: review+
Landed the changes in l10n-central: http://hg.mozilla.org/l10n-central/kn/rev/2e93b2fa06fb
Depends on: 615177
lsv updated in https://hg.mozilla.org/l10n/l10n-src-verification/rev/2ca70f9f4261.

Not closing the bug yet, waiting for bug 615177 to be resolved first.
Whiteboard: [update l10n-src-verification] → [l10n-src-verification updated]
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: