xul:menuitem should have a way to get parent xul:menu

RESOLVED FIXED

Status

()

Toolkit
XUL Widgets
RESOLVED FIXED
11 years ago
10 years ago

People

(Reporter: surkov, Assigned: surkov)

Tracking

({dev-doc-complete})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

5.33 KB, patch
Neil Deakin
: review+
neil@parkwaycc.co.uk
: review+
Mike Schroepfer
: approval1.9+
Details | Diff | Splinter Review
(Assignee)

Description

11 years ago
This is spun off bug 372552. Now xul:menu has ability to navigate via xul:menu hierarchy and it is helful for accessibility code. Now I need to get an ability to get xul:menu (nsIDOMXULContainerElement) from xul:menuitem. Now I can see one way, it is to add interface like nsIDOMXULContainerItemElement.

interface nsIDOMXULContainerItemElement: public nsIDOMXULElement{
  readonly attribute nsIDOMXULContainerElemenet container;
};

What is the best way?
(Assignee)

Updated

11 years ago
Blocks: 389926
(Assignee)

Comment 1

10 years ago
Would it be ok if menuitem will implement new nsIDOMXULContainerItemElement  interface from the comment above?

Comment 2

10 years ago
Yes, if the method is 'parentContainer'.
(Assignee)

Comment 3

10 years ago
Created attachment 301281 [details] [diff] [review]
patch
Assignee: nobody → surkov.alexander
Status: NEW → ASSIGNED
Attachment #301281 - Flags: review?
(Assignee)

Updated

10 years ago
Attachment #301281 - Flags: review? → review?(enndeakin)

Updated

10 years ago
Attachment #301281 - Flags: review?(enndeakin) → review+
(Assignee)

Updated

10 years ago
Attachment #301281 - Flags: review?(neil)

Comment 4

10 years ago
Comment on attachment 301281 [details] [diff] [review]
patch

sr=me
Attachment #301281 - Flags: review?(neil) → review+
(Assignee)

Updated

10 years ago
Attachment #301281 - Flags: approval1.9?

Updated

10 years ago
Attachment #301281 - Flags: approval1.9? → approval1.9+
(Assignee)

Comment 5

10 years ago
checked in
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
(Assignee)

Updated

10 years ago
Keywords: dev-doc-needed
Documented.
Keywords: dev-doc-needed → dev-doc-complete
You need to log in before you can comment on or make changes to this bug.