355
edits
Labs/Bespin/DesignDocs: Difference between revisions
no edit summary
No edit summary |
|||
| Line 67: | Line 67: | ||
* [[Labs/Bespin/DesignDocs/Inline Documentation|Overview]] | * [[Labs/Bespin/DesignDocs/Inline Documentation|Overview]] | ||
== Refactoring / Cleanup == | |||
It may not be sexy, but it gotta get done... and you get a great feeling when you make something better! | |||
Here's the list: | |||
* [[Labs/Bespin/DesignDocs/Refactoring/FixEventing|Fix Event System Usage]]: Pub sub is awesome, but we use it for too much. Keep items that truly are "this happened" but move out the "do this" and make real method calls | |||
* [[Labs/Bespin/DesignDocs/Refactoring/KeyHandler|Key Handling Component]]: Refactor the key handling out of editor.js and other sprinklings and put into one component (works with focus handling) | |||
* [[Labs/Bespin/DesignDocs/Refactoring/FocusHandler|Focus Handling Component]]: One focus handler that you can register with | |||
* Conventions: | |||
** [[Labs/Bespin/DesignDocs/Refactoring/JavaScriptLocations|JavaScript File Locations]]: Instead of module/module.js, move back to module.js which can include subfiles from module/*.js. | |||
** [[Labs/Bespin/DesignDocs/Refactoring/Hitching|Code Hitching]]: Rules for when you should dojo.hitch() versus "var self" and clean up code to do it | |||
== Fun == | == Fun == | ||