r105221 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r105220‎ | r105221 | r105222 >
Date:20:42, 5 December 2011
Author:tparscal
Status:deferred
Tags:
Comment:
Moved es.Tool to base classes
Modified paths:
  • /trunk/extensions/VisualEditor/VisualEditor.php (modified) (history)
  • /trunk/extensions/VisualEditor/demo/index.html (modified) (history)
  • /trunk/extensions/VisualEditor/modules/es/bases/es.Tool.js (added) (history)
  • /trunk/extensions/VisualEditor/modules/es/tools/es.Tool.js (deleted) (history)

Diff [purge]

Index: trunk/extensions/VisualEditor/VisualEditor.php
@@ -73,6 +73,7 @@
7474 'es/bases/es.DocumentViewNode.js',
7575 'es/bases/es.DocumentViewBranchNode.js',
7676 'es/bases/es.DocumentViewLeafNode.js',
 77+ 'es/bases/es.Tool.js',
7778 'es/models/es.DocumentModel.js',
7879 'es/models/es.HeadingModel.js',
7980 'es/models/es.ListItemModel.js',
@@ -102,7 +103,6 @@
103104 'es/views/es.TableRowView.js',
104105 'es/views/es.TableView.js',
105106 'es/views/es.ToolbarView.js',
106 - 'es/tools/es.Tool.js',
107107 'es/tools/es.ButtonTool.js',
108108 'es/tools/es.AnnotationButtonTool.js',
109109 'es/tools/es.ClearButtonTool.js',
Index: trunk/extensions/VisualEditor/demo/index.html
@@ -104,6 +104,7 @@
105105 <script src="../modules/es/bases/es.DocumentViewNode.js"></script>
106106 <script src="../modules/es/bases/es.DocumentViewBranchNode.js"></script>
107107 <script src="../modules/es/bases/es.DocumentViewLeafNode.js"></script>
 108+ <script src="../modules/es/bases/es.Tool.js"></script>
108109
109110 <!-- Models -->
110111 <script src="../modules/es/models/es.SurfaceModel.js"></script>
@@ -134,7 +135,6 @@
135136 <script src="../modules/es/views/es.TableCellView.js"></script>
136137 <script src="../modules/es/views/es.HeadingView.js"></script>
137138
138 - <script src="../modules/es/tools/es.Tool.js"></script>
139139 <script src="../modules/es/tools/es.ButtonTool.js"></script>
140140 <script src="../modules/es/tools/es.AnnotationButtonTool.js"></script>
141141 <script src="../modules/es/tools/es.ClearButtonTool.js"></script>
Index: trunk/extensions/VisualEditor/modules/es/tools/es.Tool.js
@@ -1,11 +0,0 @@
2 -es.Tool = function( toolbar, name ) {
3 - this.toolbar = toolbar;
4 - this.name = name;
5 - this.$ = $( '<div>' ).attr( 'title', this.name );
6 -};
7 -
8 -es.Tool.prototype.updateState = function() {
9 - throw 'Tool.updateState not implemented in this subclass:' + this.constructor;
10 -};
11 -
12 -es.Tool.tools = {};
Index: trunk/extensions/VisualEditor/modules/es/bases/es.Tool.js
@@ -0,0 +1,11 @@
 2+es.Tool = function( toolbar, name ) {
 3+ this.toolbar = toolbar;
 4+ this.name = name;
 5+ this.$ = $( '<div>' ).attr( 'title', this.name );
 6+};
 7+
 8+es.Tool.prototype.updateState = function() {
 9+ throw 'Tool.updateState not implemented in this subclass:' + this.constructor;
 10+};
 11+
 12+es.Tool.tools = {};
Property changes on: trunk/extensions/VisualEditor/modules/es/bases/es.Tool.js
___________________________________________________________________
Added: svn:eol-style
113 + native

Status & tagging log