64449 - Move text zoom menu into its own overlay

Status: VERIFIED FIXED

(SeaMonkey :: UI Design, defect)

Description

[Peter "jag" Annema]

I'll use this bug to track moving the text zoom menu into its own overlay so it
can be used by navigator and mailnews (and possibly composer).

Comment 1

[Peter "jag" Annema]

Attachment: [patch] Move text zoom menu into its own overlay.

Comment 2

[Peter "jag" Annema]

In stringbundleBindings.xml, for the stringbundleset, change
extends="xul:spring" to extends="xul:box". (It needs to be a container).

Comment 3

[Ben Goodger (use ben at mozilla dot org for email)]

a=[e-mail]

Comment 4

[Peter "jag" Annema]

Attachment: [patch] few small changes based on timeless' and moa's feedback.

Comment 5

[timeless]

r=timeless

Comment 6

[Peter "jag" Annema]

Checked in, marking fixed.

Comment 7

[Peter "jag" Annema]

Minor fix needed: in viewZoomOverlay.js, the comment about <menuitem
id="view_textZoomMenu"/> should be <menu id="menu_textZoom"/>

Comment 8

[sairuh (rarely reading bugmail)]

checked lxr: both viewZoomOverlay.js and viewZoomOverlay.xul now exist, and
askViewZoom.js and askViewZoom.xul are now in
xpfe/communicator/resources/content/ .