Bug 604013 (fx40-p12n-sr)

[sr] Firefox 4.0 productization review for Serbian

RESOLVED WONTFIX

Status

Mozilla Localizations
sr / Serbian
RESOLVED WONTFIX
8 years ago
7 years ago

People

(Reporter: stas, Assigned: Filip Miletic)

Tracking

({productization})

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [update l10n-src-verifcation])

Attachments

(2 attachments)

(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 Serbian 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/sr>

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 Serbian 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

7 years ago
Bloglines still remains in your 1.9.2 repository.  Please prepare a patch to remove it. 

Also, 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.  Thank you.
(Assignee)

Updated

7 years ago
Assignee: nobody → filmil

Comment 2

7 years ago
Ping.  Are you guys preparing a patch to remove Bloglines?  And, are there any productization changes you are considering for Firefox 4?
(Assignee)

Comment 3

7 years ago
> Ping.  Are you guys preparing a patch to remove Bloglines?  And, are there any
> productization changes you are considering for Firefox 4?

Seth, this has been on my radar for far too long.  I'll get to it asap.

Comment 4

7 years ago
Filip, any word on this?  Need to close this soon.
(Assignee)

Comment 5

7 years ago
(In reply to comment #4)
> Filip, any word on this?  Need to close this soon.

Seth, talking about soon: how soon?  If you need this closed, I'll get to it ASAP.  What's the deadline?

To answer your question, no word yet.  If this is a bottleneck somewhere, I'll wrap it up.  My private priority for resolving this is low, as the SR ff beta 4   is a mess that needs careful repair, and I've grown a bit demotivated having to correct one and the same problem so many times over.

Comment 6

7 years ago
Very soon.  We are expecting a release candidate in 10 days.  I am sorry to hear about the Serbian FF 4 beta and have followed it lightly in an email thread.  I don't know the exact state of the issue right now, but if I remember, you had a problem with the access keys?  Is there any community who can help you on this?
(Assignee)

Comment 7

7 years ago
Created attachment 514729 [details] [diff] [review]
Removes Bloglines from 1.9.2
Attachment #514729 - Flags: review?(stas)
(Assignee)

Comment 8

7 years ago
(In reply to comment #0)

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

Not there, considering solved.

> 2) Bloglines is closing down on November 1st and we need to remove it from the

- Made a patch for 1.9.2.

- Default resolution for central (mass-land with reader first) is OK.

- Adding bing to list.txt is OK, patch will follow.
(Assignee)

Comment 9

7 years ago
Created attachment 514733 [details] [diff] [review]
Adds bing to l10n-central for Serbian

As instructed, adding bing only to list.txt.
Attachment #514733 - Flags: review?(stas)
(Assignee)

Updated

7 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 10

7 years ago
If I understand correctly, the two patches round off the product review for 4.0.

Pending approval, I'll submit these and close the bug.

Comment 11

7 years ago
Comment on attachment 514729 [details] [diff] [review]
Removes Bloglines from 1.9.2

Looks good.  Please land on 1.9.2.  Add a comment that references this bug and the change.  After you have landed, please test and then use Pike's sign off application to sign off on your good 1.9.2 build.
Attachment #514729 - Flags: review?(stas) → review+

Comment 12

7 years ago
Comment on attachment 514733 [details] [diff] [review]
Adds bing to l10n-central for Serbian

Land on l10n-central and add a comment to the check in that references this bug, r=sethb, and the change.  After you push this to your hg repo, please test, and then sign off on the successful build using Pike's sign off app.  Then, you can resolve this bug as fixed.
Attachment #514733 - Flags: review?(stas) → review+

Updated

7 years ago
Whiteboard: [update l10n-src-verifcation]
(Assignee)

Comment 13

7 years ago
Thanks.   I haven't closed this yet as the Serbian repo is pending some radical changes and hasn't produced a satisfactory build yet, since my last post.
(Assignee)

Comment 14

7 years ago
The patches have long been landed to the repo, but the work on l10n-central is over.  

Obsoleting.
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.