Opened 13 years ago

Closed 13 years ago

Last modified 12 years ago

#2802 closed defect (fixed)

Menu.js replace getElementsByClass code

Reported by: Adam Peller Owned by: simonjb
Priority: high Milestone:
Component: Widgets Version: 0.4.2
Keywords: Cc: bill, Adam Peller
Blocked By: Blocking:

Description

From Bill's e-mail, regarding Menu.js:

Looks like _getElementsByClass() definition isn't necessary.   Instead of

       _getElementsByClass("dojoMenuItemLabel", item.domNode)[0];

Can just do
       item.containerNode

Change History (5)

comment:1 Changed 13 years ago by alex

or use dojo.query() since it's always available

comment:2 Changed 13 years ago by davidb

Owner: changed from davidb to simonjb

comment:3 Changed 13 years ago by simonjb

Status: newassigned

comment:4 Changed 13 years ago by bill

Resolution: fixed
Status: assignedclosed

(In [8379]) Remove unused function; fixes #2802

comment:5 Changed 12 years ago by (none)

Milestone: 0.9M2

Milestone 0.9M2 deleted

Note: See TracTickets for help on using tickets.