Closed Bug 961994 Opened 10 years ago Closed 10 years ago

Move aboutReader.html to /toolkit

Categories

(Toolkit :: General, defect, P1)

x86_64
All
defect

Tracking

()

RESOLVED DUPLICATE of bug 793920

People

(Reporter: eklavyamirani, Assigned: eklavyamirani)

References

Details

(Whiteboard: p=0)

Attachments

(1 file, 1 obsolete file)

Readability parsed pages are shown in about:reader (reader mode).

As per the reviews in Bug 927124 and Bug 558882, aboutReader.html and related content should be moved to /toolkit/components/reader/
Assignee: nobody → eklavyamirani
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Follows after the patch in Bug 927124

I am still unsure about which css goes in toolkit and which is platform specific, so I just copied the metro one to toolkit and then overrid the entire css in toolkit with the metro one. (tested without the override, it works)
Attachment #8363235 - Flags: review?(bmcbride)
Sorry for the last one, got a few things messed up there.

Moved aboutReader.html and its dependencies to toolkit
Reader mode now works on firefox desktop and metro firefox (the patch in Bug 558882 can be applied over this.)
aboutReader.css can be overriden, but assuming we need two stylesheets, I am using the default one (Not sure what needs to be in either of the stylesheets)

Strangely, if I try to load a stylesheet from the same directory as aboutReader.html, it gives me a security error. Why is that?
Attachment #8363235 - Attachment is obsolete: true
Attachment #8363235 - Flags: review?(bmcbride)
Attachment #8365708 - Flags: review?(bmcbride)
Whiteboard: p=0
Attachment #8365708 - Flags: review?(bmcbride)
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: