All users were logged out of Bugzilla on October 13th, 2018

Firefox for iOS (Build 22): "Send Tab" appears twice and not localized

RESOLVED FIXED

Status

()

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: flod, Unassigned)

Tracking

unspecified
Other
iOS

Firefox Tracking Flags

(fxios+)

Details

Attachments

(3 attachments)

(Reporter)

Description

3 years ago
Created attachment 8624204 [details]
Screenshot build22

+++ This bug was initially created as a clone of Bug #1157241 +++

Send Tab is currently appearing twice and not localized.
I don't see it twice on my iPhone 5s, only unlocalized.
tracking-fxios: --- → ?
Assignee: nobody → sarentz
Status: NEW → ASSIGNED
tracking-fxios: ? → +
Assignee: sarentz → nobody
Status: ASSIGNED → NEW
There are two Send Tab strings:

Extensions/SendTo/Info.plist
Extensions/SendTo/SendTo.strings

The first is the name of the Send Tab action in the Share menu.
The second is the title of the modal view that shows up when you select the Send Tab action.

I can provide screenshots of both. Let me know.

I'm looking into why this is not appearing localized.
Flags: needinfo?(splewako)
Flags: needinfo?(francesco.lodolo)
(Reporter)

Comment 3

3 years ago
Not sure what the question is. I just tried with simulator:
* If I import the Italian XLIFF file in XCode, I have one Send tab and it's perfectly localized.
* If I import using scripts/import-locales.sh, same result.

But strings on SVN changed in the last week, so not sure if that's the reason it got fixed from the current Testflight build.
Flags: needinfo?(francesco.lodolo)
Assignee: nobody → sarentz
Status: NEW → ASSIGNED
:flod I am confused. What did you see correctly localized? The title of the dialog or the title under the button/icon in the (system-provided) Share menu?

(I'm pretty sure the action title in the share menu is not localized - i almost have a patch for that)
Flags: needinfo?(francesco.lodolo)
(Reporter)

Comment 5

3 years ago
Created attachment 8626822 [details]
Simulator screenshot

Both if I understand the question. The only weirdness is the first device being identified as $(MOZ_BUNDLE_DISPLAY_NAME) in the list of devices
Flags: needinfo?(francesco.lodolo)
Created attachment 8626823 [details] [review]
PR: https://github.com/mozilla/firefox-ios/pull/642

TL;DR The Send Tab action title is now correctly localized when presented in the system-provided Share sheet.

This patch fixes a couple of things:

* We never correctly converted Info.plist sections in the XLIFF to a `.strings` file. They would end up in the project as `XX.lproj/Info.plist`, which is wrong. They are now written to `XX.lproj/InfoPlist.strings` so that the strings are correctly picked up by iOS.
* The `xliff-cleanup.py` script was broken and I think it never correctly processed `Info.plist` sections anyway.
* The l10n scripts normally ignore the `CFBundleDisplayName` but we now make an exception if that key is found in `Extensions/SendTo/Info.plist`
Attachment #8626823 - Flags: review?(francesco.lodolo)
I don't understand why that worked for you. It should not work. It most certainly did not work for me in Dutch :-/
This patch also makes sure the NSLocationWhenInUseUsageDescription key is output to Client/XX.lproj/InfoPlist.strings. I don't think it correctly worked before.
(Reporter)

Comment 9

3 years ago
(In reply to Stefan Arentz [:st3fan] from comment #7)
> I don't understand why that worked for you. It should not work. It most
> certainly did not work for me in Dutch :-/

Are those strings translated for Dutch? Maybe it's same weird XCode caching from when I tried to import the file from within XCode (but I also always run "git clean -fd" in the repo).
(Reporter)

Comment 10

3 years ago
Comment on attachment 8626823 [details] [review]
PR: https://github.com/mozilla/firefox-ios/pull/642

Freaking XCode cache... Did Product->Clean and the button was in English. Patched version works as expected.
Flags: needinfo?(splewako)
Attachment #8626823 - Flags: review?(francesco.lodolo) → review+
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Assignee: sarentz → nobody
You need to log in before you can comment on or make changes to this bug.