Closed Bug 57984 Opened 24 years ago Closed 24 years ago

make nsIContentViewer scriptable

Categories

(SeaMonkey :: UI Design, defect, P3)

defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: jag+mozbugs, Assigned: jag+mozbugs)

References

Details

To further our efforts to get rid of appCore, we'll need to have access to the
textZoom property of nsIMarkupDocumentViewer from both Editor and Browser.

This can be gotten at through the documentViewer property of a DocShell, except
right now it's not scriptable.

See here the bug with a request to make it scriptable.

Not sure if XP Apps is the right component, please re-assign if not.

I'll see if I can do this (more hands-on learning!) but if anyone else wants to
do this, please do :-)
Blocks: 54964
*cough* nsIContentViewer *cough* contentViewer property *cough*
Summary: make nsIDocumentViewer scriptable → make nsIContentViewer scriptable
Blocks: 10893
Oooh, and let's not forget about mail/news. HTML e-mails will also benefit from
this zoom fu (bug 10893) :-)
Blocks: 46200
According to bryner, the only thing for nsIContentViewer which can't be
scriptable is:

NS_IMETHOD Init(nsIWidget* aParentWidget,
                nsIDeviceContext* aDeviceContext,
                const nsRect& aBounds)=0;

because of the nsIDeviceContext.

Currently I'll only need to get the contentViewer from a docShell and most of
the time just QI it to another interface, so that shouldn't be a problem I
think.
Taking this bug... It works here! I'll attach a patch tomorrow.
Assignee: don → disttsc
Done! See bug 46200 for the patch to this and its little friends.
Status: NEW → ASSIGNED
And checked in (a while back)
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
rubberstamp vrfy.
Status: RESOLVED → VERIFIED
Product: Core → Mozilla Application Suite
You need to log in before you can comment on or make changes to this bug.