Improve strings and l10n comment in app-manager.properties

RESOLVED FIXED in Firefox 26

Status

RESOLVED FIXED
5 years ago
3 months ago

People

(Reporter: flod, Assigned: flod)

Tracking

Trunk
Firefox 26
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
The current file is completely missing localization comments and has at least 2 strings with problems.

validator.invalidManifestJSON=The webapp manifest isn't a valid JSON file: %1$S at: %2$S

If I read the code right, variable should be swapped (%1$S is the url, %2$S the error message).

validator.missIconForMarketplace=app submission to the Marketplace needs at least an 128 icon

"an 128 icon" doesn't sound right to me, I guess adding pixel would make sense too.
(Assignee)

Comment 1

5 years ago
Created attachment 800640 [details] [diff] [review]
bug913366v1.patch
Attachment #800640 - Flags: review?(paul)

Comment 2

5 years ago
Comment on attachment 800640 [details] [diff] [review]
bug913366v1.patch

Thank you Francesco.
Attachment #800640 - Flags: review?(paul) → review+
(Assignee)

Updated

5 years ago
Keywords: checkin-needed

Updated

5 years ago
Whiteboard: [land-in-fx-team]
https://hg.mozilla.org/integration/fx-team/rev/e18393335796
Keywords: checkin-needed
Whiteboard: [land-in-fx-team] → [land-in-fx-team][fixed-in-fx-team]

Updated

5 years ago
Whiteboard: [land-in-fx-team][fixed-in-fx-team] → [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/e18393335796
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 26

Updated

3 months ago
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.