Opened 8 years ago

Last modified 8 years ago

#13190 closed defect

dijit.CalendarLite - should _onMonthSelectbe using this._setCurrentFocusAttr — at Version 1

Reported by: ben hockey Owned by:
Priority: high Milestone: 1.7
Component: Dijit Version: 1.7.0b1
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by ben hockey)

_onMonthSelect assigns to this.currentFocus rather than calling this._setCurrentFocusAttr. when the user changes month via incrementMonth or decrementMonth nodes, this._setCurrentFocusAttr is used. the inconsistency means that a watch on currentFocus misses changes made via _onMonthSelect. would it be ok to change _onMonthSelect to call this._setCurrentFocus?

the code is only used by dijit.Calendar with the dropdown for the month. maybe the function could also move to dijit.Calendar

Change History (1)

comment:1 Changed 8 years ago by ben hockey

Description: modified (diff)
Note: See TracTickets for help on using tickets.