landing neil's fix for bug #196755

[mailviews] [modern skin only] Unable to create new Views (pull down menus do not retain changes)
r=jkeiser, sr=jag
This commit is contained in:
sspitzer%netscape.com 2003-03-23 17:38:51 +00:00
parent 62dca39a90
commit fe7206cf7a

View File

@ -24,12 +24,20 @@
</content>
<handlers>
<handler event="command" phase="capturing"
action="if (event.target.parentNode.parentNode == this) this.selectedItem = event.target;"/>
<handler event="command" phase="capturing">
<![CDATA[
// XXX see bug #196755
// originalTarget is workaround for arrowscrollbox retargeting bug
if (event.originalTarget.parentNode.parentNode == this)
this.selectedItem = event.originalTarget;
]]>
</handler>
<handler event="popupshowing">
<![CDATA[
if (event.target.parentNode == this && this.selectedItem)
// XXX see bug #196755
// originalTarget is workaround for arrowscrollbox retargeting bug
if (event.originalTarget.parentNode == this && this.selectedItem)
// Not ready for auto-setting the active child in hierarchies yet.
// For now, only do this when the outermost menupopup opens.
this.menuBoxObject.activeChild = this.selectedInternal;
@ -402,7 +410,9 @@
<![CDATA[
// editable menulists elements aren't in the focus order,
// so when the popup opens we need to force the focus to the inputField
if (event.target.parentNode == this) {
// XXX see bug #196755
// originalTarget is workaround for arrowscrollbox retargeting bug
if (event.originalTarget.parentNode == this) {
if (document.commandDispatcher.focusedElement != this.inputField)
this.inputField.focus();