Look for immediate children of popup notification icon box as anchors

RESOLVED FIXED in mozilla2.0b5

Status

()

Toolkit
XUL Widgets
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: Mardak, Assigned: Mardak)

Tracking

Trunk
mozilla2.0b5
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Comment 1

8 years ago
Created attachment 466961 [details] [diff] [review]
v1
Assignee: nobody → edilee
Status: NEW → ASSIGNED
Attachment #466961 - Flags: review?(gavin.sharp)
Add a test for this to browser_popupNotifications?
(Assignee)

Comment 3

8 years ago
Created attachment 466973 [details] [diff] [review]
v2
Attachment #466961 - Attachment is obsolete: true
Attachment #466961 - Flags: review?(gavin.sharp)
Attachment #466973 - Flags: review?(gavin.sharp)
(Assignee)

Updated

8 years ago
Blocks: 571409
Comment on attachment 466973 [details] [diff] [review]
v2

>diff --git a/browser/base/content/test/browser_popupNotification.js b/browser/base/content/test/browser_popupNotification.js

>+  { // Test #14
>+    run: function() {

>+      EventUtils.synthesizeMouse(button, 0, 0, {});

1,1 ?
Attachment #466973 - Flags: review?(gavin.sharp) → review+
(Assignee)

Comment 5

8 years ago
Created attachment 468146 [details] [diff] [review]
v2.1

Account manager uses some complex icons that need this.
Attachment #466973 - Attachment is obsolete: true
Attachment #468146 - Flags: approval2.0?
Attachment #468146 - Flags: approval2.0? → approval2.0+
(Assignee)

Comment 6

8 years ago
http://hg.mozilla.org/mozilla-central/rev/1b7a4cadf7d1
Walk up the parent of the click target until we're just inside the icon box.
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b5
You need to log in before you can comment on or make changes to this bug.