Closed Bug 136912 Opened 22 years ago Closed 22 years ago

Context menus don't work in editable menulists

Categories

(Core :: XUL, defect)

Product:
Core
Component:
XUL
Platform:
x86
All
Type:
defect
Priority:
Not set
Severity:
major

Tracking

()

Status:
RESOLVED DUPLICATE of bug 118038

People

(Reporter: jon2121, Assigned: jag+mozilla)

References

Details

(Keywords: regression, relnote) 

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Win98; en-US; rv:0.9.9) Gecko/20020311
BuildID:     en-US; rv:0.9.9) Gecko/20020311

In this version of composer:

In the insert link command the link location field in the link properties box
will no longer allow you to paste a link address.  In fact, the right click menu
is completely disabled.  The link text field above will allow you to paste a name.

Reproducible: Always
Steps to Reproduce:
1.open composer
2.click insert/link
3.right click link address field

Actual Results:  right click disabled 

Expected Results:  right click menu with copy, paste etc..
using 4/10 build this worksforme..I can paste in an address
using the context menu inside the link props dialog.

retry in newer build...
duplicated, over to cmanske
Assignee: syd → cmanske
Target Milestone: --- → Future
Is the problem only in one of the edit fields or do all edit fields in that
dialog have a problem?  Is the problem specific to the link dialog or do other
dialogs also have this problem?

Confirm based on Syd's comment
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: regression
Summary: right click disabled in "insert link location box → right click disabled in "insert link" location box
Target Milestone: Future → mozilla1.1beta
This bug is still not fixed in:

Mozilla 1.0 Release Candidate 1
Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.0rc1) Gecko/20020417

[Ctrl] [v] will allow you to paste the link, but right click is still disabled.
Jon Bain--can you paste it into an image dialog?  I'm trying to understand if
the context menu is broken only for the link dialog or for all dialogs.
The problem is that context menus don't work in "editable menulists", even
though there is an input field inside. Today, I don't see any popup menu at all,
not just menuitems that are disabled!
Kin: Didn't you recently fix context popups in textfields? The textfield in 
and editable menulist is annonymous countent inside the <menulist> widget.

Context menus now work correctly in <textbox> XUL widget, but not in 
<menulist editable="true">.
Obviously some XBL binding magic needs to happen to make them work in the 
input portion that is anonymous content inside the menulist widget.
Assignee: cmanske → jaggernaut
Component: Editor: Composer → XP Toolkit/Widgets
QA Contact: sujay → jrgm
Summary: right click disabled in "insert link" location box → Context menus don't work in editable menulists
*** Bug 146052 has been marked as a duplicate of this bug. ***
*** Bug 154343 has been marked as a duplicate of this bug. ***
This sucks. Unfortuntaley, it affects a very important feature: pasting 
a link inside Composer's Link Properties dialog.
We definitely need to release note this.
cc Robin so she can release note
Keywords: relnote
*** Bug 164017 has been marked as a duplicate of this bug. ***
This bug is one of the top complaints we get for Composer.  Nominating for
nsbeta1 consideration.  Reset milestone to current milestone (but it should
probably be retriaged).
Keywords: nsbeta1
OS: Windows 98 → All
Target Milestone: mozilla1.1beta → mozilla1.2final
*** Bug 177481 has been marked as a duplicate of this bug. ***
Jan, Blake, could either of you take a look at this?

Clearing the milestone since no commitment to fixing this has been made yet.
Target Milestone: mozilla1.2final → ---
I think we should add "textbox-input-box" class here:
  <binding id="menulist-editable" extends="..."
    <content>
      <xul:hbox class="textbox-input-box menulist-editable-box" flex="1">
        <html:input flex="1" class="menulist-editable-input" allowevents="true"
                    xbl:inherits="value=label,disabled"/>
      </xul:hbox>

but there's more to do, since now there are 2 menupopups, so code needs to count
with it

Nav triage team: nsbeta1-
Keywords: nsbeta1 → nsbeta1-
This is working in my build as of 12/5/02.
Fixed by Neil's fix for bug 118038 (or some other bug he's fixed recently)?
Neil can dup this.


*** This bug has been marked as a duplicate of 118038 ***
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.