Opened 7 years ago
Closed 6 years ago
#15520 closed enhancement (fixed)
[patch][cla] dojox/widget/Calendar doesn't highlight month or year
| Reported by: | dylan | Owned by: | dylan |
|---|---|---|---|
| Priority: | low | Milestone: | 1.9 |
| Component: | DojoX Widgets | Version: | 1.7.2 |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: |
Description
dojox/widget/Calendar has apparently never supported a mechanism for setting a style for the currently selected month. This should be set in _CalendarYearView.js and _CalendarMonthView.js similar to how it's set in _calendarDayView.js, but does not appear to have ever been implemented (I checked back to Dojo 1.6).
Attachments (3)
Change History (10)
comment:1 Changed 6 years ago by
| Status: | new → assigned |
|---|
comment:2 Changed 6 years ago by
| Milestone: | 2.0 → 1.9 |
|---|
So, you probably meant to set the milestone to 1.9?
Changed 6 years ago by
| Attachment: | patch_15520.0.diff added |
|---|
Style current month / year just as current day is styled on dojox/widget/Calandar
comment:4 Changed 6 years ago by
First pass at a patch to style the current month / year attached. Verified using dojox/widget/tests/test_CalendarViews.html.
comment:5 Changed 6 years ago by
This one should be good now. Both the CalendarMonthView and CalendarYearView now support proper styling of the selected month / year respectively. Existing CSS classes were used.
Changed 6 years ago by
| Attachment: | patch_15520.2.diff added |
|---|
Declare variables one level higher in the query
comment:7 Changed 6 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
| Summary: | dojox/widget/Calendar doesn't highlight month or year → [patch][cla] dojox/widget/Calendar doesn't highlight month or year |
Thanks for the refinements and patch @bitpshr. Committed to trunk.
Will try to fix this for 1.9.