Bug 603984 (fx40-p12n-kn)

[kn] Firefox 4.0 productization review for Kannada

RESOLVED FIXED

Status

Mozilla Localizations
kn / Kannada
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: stas, Unassigned)

Tracking

({productization})

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [l10n-src-verification updated])

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

8 years ago
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!

Comment 1

8 years ago
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.

Comment 2

8 years ago
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.

Comment 3

8 years ago
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?

Comment 4

8 years ago
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

Comment 5

8 years ago
By the way, why this bug was not listed in webdashboard?

Probably, that was the reason I missing this!

Shankar

Comment 6

8 years ago
I have also added bing to the list.txt

Shankar

Comment 7

8 years ago
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

Comment 8

8 years ago
Created attachment 492592 [details] [diff] [review]
patch for region.properties

Here is the patch file for region.properties

Shankar

Comment 9

8 years ago
Created attachment 492593 [details] [diff] [review]
patch for list.txt

Here is the the patch for list.txt

Updated

8 years ago
Attachment #492592 - Attachment is patch: true
Attachment #492592 - Attachment mime type: application/octet-stream → text/plain

Comment 10

8 years ago
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)

Updated

8 years ago
Attachment #492593 - Flags: review?(sethb)

Comment 11

8 years ago
There is no particular reason as such. I just copied the content from mozilla-central! :)

Comment 12

8 years ago
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 13

8 years ago
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+

Updated

8 years ago
Whiteboard: [update l10n-src-verification]

Comment 14

8 years ago
Created attachment 493371 [details] [diff] [review]
patch for region.properties

I have included netvibes in this patch
Attachment #492592 - Attachment is obsolete: true

Comment 15

8 years ago
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?

Comment 16

8 years ago
/me edits, hang on

Updated

8 years ago
Attachment #493371 - Attachment is patch: true
Attachment #493371 - Attachment mime type: application/octet-stream → text/plain

Comment 17

8 years ago
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.

Comment 18

8 years ago
Thanks, Pike.  Didn't know that.  :)

Comment 19

8 years ago
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+

Comment 20

8 years ago
Landed the changes in l10n-central: http://hg.mozilla.org/l10n-central/kn/rev/2e93b2fa06fb
(Reporter)

Updated

8 years ago
Depends on: 615177
(Reporter)

Comment 21

8 years ago
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]
(Reporter)

Updated

8 years ago
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.