thatcher
|
114a26b1a9
|
found some lingering references to 'this' in private closure methods.
|
2011-12-20 07:44:33 -05:00 |
|
thatcher
|
345e5f3e6c
|
completed first pass at refactor of buttons.js, next will be mousetracker so we can attack some significant anti-patterns that spill over into places like buttons.js because the MouseTracker doesnt have an idiomatic constructor. It may also be worth applying the MouseTracker as a mixin to avoid the extra .tracker property indirection.
|
2011-12-20 07:39:02 -05:00 |
|
thatcher
|
acfd3d0280
|
removed more psuedo-private methods and properties in favor of direct accessors or closure private methods. moved raiseEvent to EventHandler interface
|
2011-12-16 18:29:16 -05:00 |
|
thatcher
|
1395157adf
|
removed this._button from Button and renamed it to this.element since thats what this.get_element returned. may eventually move to a more jquery like pattern of extending the element with the methods of Button.
|
2011-12-16 17:56:38 -05:00 |
|
thatcher
|
7d4ffaa769
|
refactored EventHandlerList to just EventHandler and am prefering a mix-in on the prototype to avoid excess indirection via psuedo-private properties.
|
2011-12-14 18:22:02 -05:00 |
|
thatcher
|
01153bcb91
|
remove add_foo remove_foo methods in favor of adding them directly via this._events.addHandler etc. made some notes on how to replace this._events in general by mixing-in EventHandlerList to the object prototype so this.addHandler can be used directly.
|
2011-12-14 17:54:42 -05:00 |
|
thatcher
|
d7de748434
|
removed initialize anti-pattern in favor of using constructor. combined Button constructor arguments into idiomatic single options object
|
2011-12-14 17:40:22 -05:00 |
|
thatcher
|
916ada5f02
|
Added simple ant build. Broke main file into composite parts in src directory. Concatenated release is still included in the project as a release artifact.
|
2011-12-05 22:50:25 -05:00 |
|