Opened 14 years ago

Closed 13 years ago

#2934 closed defect (fixed)

[add|remove]Listener are superseded by connect/disconnect and should be removed

Reported by: sjmiles Owned by: sjmiles
Priority: high Milestone: 0.9beta
Component: General Version:
Keywords: Cc:
Blocked By: Blocking:

Description

Before pure dojo.connect/disconnect were merged with the DOM-aware listener logic, dojo.[add|remove]Listener were public functions.

Now, connect/disconnect are preferred and have a cleaner API (disconnect only takes one argument).

I plan to remove [add|remove]Listener from the public API because I believe they only add confusion at this point.

Change History (3)

comment:1 Changed 14 years ago by bill

(In [8482]) Change addListener() calls to connect(). Refs #2934. Still need code to collect all things that need to be disconnected on widget destroy.

comment:2 Changed 13 years ago by sjmiles

(In [8918]) Privatize add/removeListener, client code should just use connect/disconnect. Refs #2934.

comment:3 Changed 13 years ago by sjmiles

Resolution: → fixed
Status: new → closed
Note: See TracTickets for help on using tickets.