Audit string changes in fig

RESOLVED FIXED in Firefox 26

Status

()

Firefox for Android
General
P1
normal
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: lucasr, Assigned: Margaret)

Tracking

unspecified
Firefox 26
All
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: fixed-fig)

Attachments

(2 attachments)

(Reporter)

Description

4 years ago
Make sure we've updated entity names accordingly for all new and updated strings.
(Reporter)

Updated

4 years ago
Priority: -- → P1
Created attachment 790735 [details] [diff] [review]
String difference between fig and m-c in patch format (for reference)

Differences between mozilla-central and projects/fig on the two paths containing strings for Fennec.

Path: /mobile/android/locales/en-US/chrome
Files modified: aboutAddons.dtd

Path: /mobile/android/base/locales/en-US
Files modified: android_strings.dtd

I'll have a deeper look, testing the nightly from fig, but everything seems good so far.
As far as I can tell, after having checked the running app, all strings have been renamed or removed, I don't see any strings modified while maintaining the old ID.

+<!ENTITY bookmarks_title "Bookmarks">
+<!ENTITY history_title "History">

These are two of the three top titles, the other one is the existing "Reading list", which used to be a folder's name in bookmarks. Not sure if it's worth changing the id (maybe readinglist_title?).

<!ENTITY reading_list "Reading List">

How are these strings transformed to upper case?


Can't find the following strings in the interface (version 2013-08-13)

+<!ENTITY all_pages_title "Top Sites">
+<!ENTITY contextmenu_remove "Remove"> (I get only Edit on context menu)
+<!ENTITY bookmark_add "Add a bookmark">
+<!ENTITY home_last_tabs_title "Tabs from last time">
+<!ENTITY home_last_tabs_open "Open all tabs from last time">

The first one I can't even find in the code except for a couple of declarations.
(Reporter)

Comment 3

4 years ago
(In reply to Francesco Lodolo [:flod] from comment #2)
> As far as I can tell, after having checked the running app, all strings have
> been renamed or removed, I don't see any strings modified while maintaining
> the old ID.
> 
> +<!ENTITY bookmarks_title "Bookmarks">
> +<!ENTITY history_title "History">
> 
> These are two of the three top titles, the other one is the existing
> "Reading list", which used to be a folder's name in bookmarks. Not sure if
> it's worth changing the id (maybe readinglist_title?).
> 
> <!ENTITY reading_list "Reading List">

This would cause unnecessary work for the translators, no?
 
> How are these strings transformed to upper case?

This is done by the platform (usually done through Android's TransformationMethod):

http://developer.android.com/reference/android/text/method/TransformationMethod.html

> Can't find the following strings in the interface (version 2013-08-13)
> 
> +<!ENTITY all_pages_title "Top Sites">
> +<!ENTITY contextmenu_remove "Remove"> (I get only Edit on context menu)
> +<!ENTITY bookmark_add "Add a bookmark">
> +<!ENTITY home_last_tabs_title "Tabs from last time">
> +<!ENTITY home_last_tabs_open "Open all tabs from last time">
> 
> The first one I can't even find in the code except for a couple of
> declarations.

Those might be leftovers but I have to double check.
(In reply to Lucas Rocha (:lucasr) from comment #3)
> This would cause unnecessary work for the translators, no?

Yes, but the same thing happens when you change the id for other strings (e.g. when you change from awesomebar_bookmarks_title to bookmarks_title, that's a new string for tools).

I personally prefer one more string (20 vs 19, if I didn't count them wrong) to make localizers aware of the changed position/usage (and probably of the different space available).
(Assignee)

Updated

4 years ago
Assignee: nobody → margaret.leibovic
(Assignee)

Comment 5

4 years ago
(In reply to Francesco Lodolo [:flod] from comment #1)

> Path: /mobile/android/locales/en-US/chrome
> Files modified: aboutAddons.dtd

FYI, this looks like a string that was just added on m-c and needs to be merged into fig, so that isn't actually related to the work we're doing.
(Assignee)

Comment 6

4 years ago
(In reply to Francesco Lodolo [:flod] from comment #2)

> +<!ENTITY all_pages_title "Top Sites">

Yes, this is unused. I'll remove it.

> +<!ENTITY contextmenu_remove "Remove"> (I get only Edit on context menu)

This is used for the home_remove context menu item, which should be shown on bookmarks or history items in the list views. (see bug 897250)

> +<!ENTITY bookmark_add "Add a bookmark">

This is shown below empty top bookmark thumbnails, instead of a page title.

> +<!ENTITY home_last_tabs_title "Tabs from last time">

This is used to set a title for the "Tabs from last time" page in the History tab. You may only see it if you do have tabs from last time.

> +<!ENTITY home_last_tabs_open "Open all tabs from last time">

This is shown for a button on that page. You also won't see it unless you have tabs from last time listed there.
(Assignee)

Comment 7

4 years ago
Created attachment 791282 [details] [diff] [review]
patch

Here's a patch to remove the unused string and update the reading list title string.
Attachment #791282 - Flags: review?(lucasr.at.mozilla)
Attachment #791282 - Flags: feedback?(francesco.lodolo)
Comment on attachment 791282 [details] [diff] [review]
patch

Review of attachment 791282 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks Margaret, patch looks good to me.

With your help I was able to find "Remove" and "Add a bookmark", not the "tabs from the last time" (I guess I'll have to understand how to get those tabs)
Attachment #791282 - Flags: feedback?(francesco.lodolo) → feedback+
(Reporter)

Comment 9

4 years ago
Comment on attachment 791282 [details] [diff] [review]
patch

Review of attachment 791282 [details] [diff] [review]:
-----------------------------------------------------------------

Nice.
Attachment #791282 - Flags: review?(lucasr.at.mozilla) → review+
(Assignee)

Comment 10

4 years ago
https://hg.mozilla.org/projects/fig/rev/73fd388d5b93
Whiteboard: fixed-fig
https://hg.mozilla.org/mozilla-central/rev/73fd388d5b93
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 26
You need to log in before you can comment on or make changes to this bug.