230072 - xml comment before xul:menupopup makes xul:menu shrink to invisible

Status: RESOLVED WONTFIX

(Core :: XBL, defect)

Description

[gekacheka]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.6b) Gecko/20031208
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.6b) Gecko/20031208, calendar 2003092614-cal

xul:menupopup appears inside xul:menu as
   <xul:menu>
       <xul:menupopup>
             (content)
       </xul:menupopup>
   </xul:menu>

But if an xml comment appears between the <xul:menu> and <xul:menupopup> as

   <xul:menu>
       <!-- comment -->
       <xul:menupopup>
             (content)
       </xul:menupopup>
   </xul:menu>

Then the xul:menu shrinks to invisible.

An example of such a menu can be found in xbl code in 
Mozilla/chrome/calendar/content/datepicker/datepicker.xml, line 15

The datepicker normally appears as a month grid button in the Event dialog.



Reproducible: Always

Steps to Reproduce:
0. install calendar if necessary.  
   New Event dialog has month grid button to open datepicker.for start and end dates
1. as above, in datepicker.xml add xml comment inside xul:menu before xul:menupopup
2. Exit mozilla and restart, with calendar (mozilla -calendar)

Actual Results:  
Now new event dialog does not show month grid buttons 


Expected Results:  
adding an xml comment should make no difference

If the debug="true" attribute is set in the enclosing xul:hbox then it appears
that when there is no comment, there are internal boxes (spacers?), but when the
comment is added, the internal boxes do not appear (so grid icons are partially
visible only due to the debug spacing).

Comment 1

[alexander :surkov (:asurkov)]

It seems the bug 340057 is related or dublicated of this one.

Comment 2

[Boris Zbarsky [:bzbarsky]]

XBL is now disabled in Firefox (Bug 1583314) and is in the process of being removed from Gecko (Bug 1566221), so closing bugs requesting changes to its implementation as wontfix.