Closed Bug 1179422 Opened 9 years ago Closed 9 years ago

Expose third-party licenses via the settings

Categories

(Firefox for iOS :: General, defect)

Product:
Firefox for iOS
Component:
General
Platform:
Other
iOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
fxios + ---

People

(Reporter: st3fan, Assigned: st3fan)

References

(Blocks 1 open bug)

Details

(Whiteboard: noteworthy)

Attachments

(2 files)

PR: https://github.com/mozilla/firefox-ios/pull/664
47 bytes, text/x-github-pull-request
sleroux
: review+
dhenein
: feedback+
Details | Review
Screenshot 2015-07-08 10.43.23.png
86.02 KB, image/png
Details 
Show a page that contains all the licenses of projects that we depend on.
Work in progress. Adds a Credits & Acknowledgements item to the About section in the Settings. That opens a local web page that lists the licenses of all the dependencies we use.
Assignee: nobody → sarentz
Status: NEW → ASSIGNED
Darrin, how do you want this to be exposed in the settings? I have a page in progress that is very much like about:license on Desktop. What should we call it in the settings?
Flags: needinfo?(dhenein)
Comment on attachment 8628393 [details] [review]
PR: https://github.com/mozilla/firefox-ios/pull/664

Adds a Licenses item to the About section in the Settings. That will open a local (about:license) web page that shows the MPL 2.0 for the main project and also lists the licenses of all the dependencies we use.
Attachment #8628393 - Flags: review?(sleroux)
Attachment #8628393 - Flags: feedback?(dhenein)
Comment on attachment 8628393 [details] [review]
PR: https://github.com/mozilla/firefox-ios/pull/664

LGTM - I also rebased it from master to update it with the latest build changes for Xcode 6.4
Attachment #8628393 - Flags: review?(sleroux) → review+
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
NSLocalizedString note seems wrong, incorrect screenshot link maybe?
Flags: needinfo?(sarentz)
I am reopening this. Looks like this PR is not complete. It is missing changes that were made. I wonder if I forgot to push some. For example the string in settings should be "Licenses" .. and I reformatted the licenses to nicer looking HTML. Looking into what happened here.
Status: RESOLVED → REOPENED
Flags: needinfo?(sarentz)
Resolution: FIXED → ---
Screenshot for localizers
Followup fix https://github.com/mozilla/firefox-ios/commit/be54e69e3bf18d526b6dfb0c48f14bf9aa3b3c62

This includes the following changes:

* Changed "Licenses & Acknowledgements" to "Licenses"
* Updated the l10n comment and included a correct screenshot
* Updated Licenses.html to include mobile friendly copies
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Flags: needinfo?(dhenein)
Resolution: --- → FIXED
Whiteboard: noteworthy
Attachment #8628393 - Flags: feedback?(dhenein) → feedback+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: