Opened 12 years ago

Closed 12 years ago

#6528 closed enhancement (fixed)

Rewrite dojox.widget.Calendar as a set of mixins

Reported by: sos Owned by: nathan
Priority: high Milestone: 1.2
Component: DojoX Form Version: 1.1.0
Keywords: Cc:
Blocked By: Blocking:

Description

The dojox.widget.Calendar widget is currently too difficult to extend. It should be broken out into a series of mixin 'views'. This will make it possible to extend it using multiple different versions of views, and also to add different visual effects.

Attachments (2)

CalendarMixinRewrite.patch (49.1 KB) - added by sos 12 years ago.
Patch to break out the calendar into mixins
CalendarMixinRewrite2.patch (49.1 KB) - added by sos 12 years ago.
Added in some missing files

Download all attachments as: .zip

Change History (6)

Changed 12 years ago by sos

Attachment: CalendarMixinRewrite.patch added

Patch to break out the calendar into mixins

comment:1 Changed 12 years ago by dante

Owner: changed from dante to Nathan Toone

comment:2 Changed 12 years ago by sos

Any feedback on this patch? Objections, suggestions?

Thanks

Shane

comment:3 Changed 12 years ago by nathan

Owner: changed from Nathan Toone to nathan

Changed 12 years ago by sos

Attachment: CalendarMixinRewrite2.patch added

Added in some missing files

comment:4 Changed 12 years ago by Nathan Toone

Resolution: fixed
Status: newclosed

(In [13652]) fixes #6528: proxy commit for shane which splits dojox.widget.Calendar into mixins. Some minor issues still exist - but they are visual issues, and probably should be tracked in separate bugs. In IE6, the "previous" and "next" buttons are not transparent (this is due to using a different background color in the header). In Safari, the fisheye effect does not work, and the custom highlighting produces weird results. However, since these are both minor issues, I am marking this bug as fixed - and separate bugs can be opened for those issues.

Note: See TracTickets for help on using tickets.