User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1b1) Gecko/20060710 BonEcho/2.0b1 Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1b1) Gecko/20060710 BonEcho/2.0b1 On a blank tab Middle mouse click on Help --> Report Phishing Website menu item opens the report phishing website page. Reproducible: Always Steps to Reproduce: 1. Open a blank tab 2. Open Help Menu 3. Middle click "Report Phishing Website" (greyed out) Actual Results: Report phishing website page opens Expected Results: Nothing happens Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1b1) Gecko/20060710 BonEcho/2.0b1 ID:2006071003
Status: UNCONFIRMED → NEW
Ever confirmed: true
Version: unspecified → 2.0 Branch
This isn't really a regression from bug 337484, checkForMiddleClick just doesn't check whether the node it's calling oncommand for is disabled. You can see the same bug if you middle click on a disabled back button, though it only throws so it's not nearly as noticeable.
Severity: normal → minor
OS: Windows XP → All
Hardware: PC → All
Summary: Report phishing Website opens with middle click even when greyed out → Report phishing Website opens with middle click even when disabled
Created attachment 228782 [details] [diff] [review] patch checkForMiddleClick is currently used on the following elements: menuitem, menu, toolbarbutton, popup, menupopup Of these, the disabled attribute only has an effect on the first three, but checking it for =="true" shouldn't hurt in the [menu]popup case.
Assignee: nobody → gavin.sharp
Status: NEW → ASSIGNED
Attachment #228782 - Flags: review?(bugs.mano)
Comment on attachment 228782 [details] [diff] [review] patch r=mano if you file a bug on the menuitem binding only caliming to implement nsIDOMXULSelectControlElement (I'm not sure if this is the right interface to implement, but we sure need the disabled property implemented when the attribute affects the element behavior). Add a comment above the check referring to that bug.
Attachment #228782 - Flags: review?(bugs.mano) → review+
I filed bug 344487.
Whiteboard: [checkin needed]
Status: ASSIGNED → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
Whiteboard: [checkin needed] → [need-a]
Target Milestone: --- → Firefox 2 beta2
Attachment #228782 - Flags: approval1.8.1?
Attachment #228782 - Flags: approval1.8.1? → approval1.8.1+
You need to log in before you can comment on or make changes to this bug.