Opened 8 years ago

Closed 7 years ago

#13264 closed enhancement (wontfix)

dojox.mvc.Group - add _patches as dependency

Reported by: ben hockey Owned by: Ed Chatelain
Priority: high Milestone: tbd
Component: DojoX MVC Version: 1.7.0b1
Keywords: Cc: rahul
Blocked By: Blocking:

Description

if i use dojox.mvc.Group as a baseclass, it doesn't automatically pull in _patches. i think it would be convenient if it would pull in _patches so that i don't have to explicitly declare another dependency (dojox/mvc or dojox/mvc/_patches) in order to have the patches applied.

i realize that once the patches are applied i may as well use _WidgetBase but it would still be 2 dependencies instead of one.

this is mostly a matter of convenience, what do you think?

Change History (3)

comment:1 Changed 8 years ago by rahul

I see the convenience. Initial assumption was most people would declare the dojox/mvc dependency, so haven't looked at such convenience in much detail yet. By the same token though, if we do this, we might as well declare the _patches dependency for other comparables such as _Container, Output etc. -- i.e. this seems more pervasive than Group?

comment:2 Changed 8 years ago by Adam Peller

Cc: rahul added
Owner: changed from rahul to Ed Chatelain

comment:3 Changed 7 years ago by Ed Chatelain

Resolution: wontfix
Status: newclosed

I am cleaning out old tickets, I don't think this is worth doing at this point.

Note: See TracTickets for help on using tickets.