Closed Bug 1127853 Opened 11 years ago Closed 9 years ago

Restructure ReaderMode.js

Categories

(Firefox for iOS :: Reader View, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: st3fan, Assigned: nalcock)

References

Details

Attachments

(1 file)

48 bytes, text/x-github-pull-request
bnicholson
: review+
Details | Review
Use the same anonymous function and window.__firefox__ setup that Client/Assets/Favicons.js does. Not super high priortity but will be nice to have since it also simplifies the code. (Ie, 'this' will actually work so no more need to prefix everything with _firefox_ReaderMode.)
Assignee: nobody → nalcock
Attached file Pull request
Attachment #8770994 - Flags: review?(sarentz)
Blocks: 1123970
Attachment #8770994 - Flags: review?(sarentz)
Attachment #8770994 - Flags: review?(bnicholson)
Comment on attachment 8770994 [details] [review] Pull request LGTM
Attachment #8770994 - Flags: review?(bnicholson) → review+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: