Skip to content

Commit e7c0792

Browse files
author
Chris K
committed
Review fixes.
1 parent de98e77 commit e7c0792

File tree

6 files changed

+41
-42
lines changed

6 files changed

+41
-42
lines changed

src/client/client.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -552,8 +552,8 @@ ui_framework.layouts.dom_rough_layout =
552552
return ['dom-side-panel',
553553
'dom_attrs',
554554
'css-layout',
555-
'dom-search',
556-
'ev-listeners-side-panel'];
555+
'ev-listeners-side-panel',
556+
'dom-search'];
557557
}
558558

559559
return ['dom-side-panel', 'dom_attrs', 'css-layout', 'dom-search'];

src/ecma-debugger/dominspection/inspectabledomnode.js

Lines changed: 30 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -34,37 +34,36 @@ cls.EcmascriptDebugger["6.0"].InspectableDOMNode.PSEUDO_NODE = 0;
3434
cls.EcmascriptDebugger["6.0"].InspectableDOMNode.prototype = new function()
3535
{
3636

37-
const
38-
NODE_LIST = 0,
39-
ID = 0,
40-
TYPE = 1,
41-
NAME = 2,
42-
DEPTH = 3,
43-
44-
ATTRS = 5,
45-
ATTR_PREFIX = 0,
46-
ATTR_KEY = 1,
47-
ATTR_VALUE = 2,
48-
CHILDREN_LENGTH = 6,
49-
PUBLIC_ID = 4,
50-
SYSTEM_ID = 5,
51-
MATCH_REASON = cls.EcmascriptDebugger["6.0"].InspectableDOMNode.MATCH_REASON,
52-
TRAVERSE_SEARCH = "search",
53-
TRAVERSAL = 1,
54-
SEARCH_PARENT = 2,
55-
SEARCH_HIT = 3,
56-
PSEUDO_TYPE = 0,
57-
PSEUDO_ELEMENT = cls.EcmascriptDebugger["6.0"].InspectableDOMNode.PSEUDO_ELEMENT,
58-
PSEUDO_NODE = cls.EcmascriptDebugger["6.0"].InspectableDOMNode.PSEUDO_NODE,
59-
BEFORE = 1,
60-
AFTER = 2,
61-
FIRST_LETTER = 3,
62-
FIRST_LINE = 4,
63-
BEFORE_ALIKES = [BEFORE, FIRST_LETTER, FIRST_LINE],
64-
AFTER_ALIKES = [AFTER],
65-
ERROR_MSG = 0,
66-
PSEUDO_NAME = {},
67-
EVENT_LISTENER_LIST = cls.EcmascriptDebugger["6.0"].InspectableDOMNode.EVENT_LISTENER_LIST;
37+
var NODE_LIST = 0;
38+
var ID = 0;
39+
var TYPE = 1;
40+
var NAME = 2;
41+
var DEPTH = 3;
42+
43+
var ATTRS = 5;
44+
var ATTR_PREFIX = 0;
45+
var ATTR_KEY = 1;
46+
var ATTR_VALUE = 2;
47+
var CHILDREN_LENGTH = 6;
48+
var PUBLIC_ID = 4;
49+
var SYSTEM_ID = 5;
50+
var MATCH_REASON = cls.EcmascriptDebugger["6.0"].InspectableDOMNode.MATCH_REASON;
51+
var TRAVERSE_SEARCH = "search";
52+
var TRAVERSAL = 1;
53+
var SEARCH_PARENT = 2;
54+
var SEARCH_HIT = 3;
55+
var PSEUDO_TYPE = 0;
56+
var PSEUDO_ELEMENT = cls.EcmascriptDebugger["6.0"].InspectableDOMNode.PSEUDO_ELEMENT;
57+
var PSEUDO_NODE = cls.EcmascriptDebugger["6.0"].InspectableDOMNode.PSEUDO_NODE;
58+
var BEFORE = 1;
59+
var AFTER = 2;
60+
var FIRST_LETTER = 3;
61+
var FIRST_LINE = 4;
62+
var BEFORE_ALIKES = [BEFORE, FIRST_LETTER, FIRST_LINE];
63+
var AFTER_ALIKES = [AFTER];
64+
var ERROR_MSG = 0;
65+
var PSEUDO_NAME = {};
66+
var EVENT_LISTENER_LIST = cls.EcmascriptDebugger["6.0"].InspectableDOMNode.EVENT_LISTENER_LIST;
6867

6968
PSEUDO_NAME[BEFORE] = "before";
7069
PSEUDO_NAME[AFTER] = "after";

src/ecma-debugger/eventlisteners/evlistenersidepanel.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ cls.EventListenerSidePanelView.create_ui_widgets = function()
1616
groups:
1717
[
1818
{
19-
type: "buttons",
19+
type: UI.TYPE_BUTTONS,
2020
items:
2121
[
2222
{
@@ -26,7 +26,7 @@ cls.EventListenerSidePanelView.create_ui_widgets = function()
2626
]
2727
},
2828
{
29-
type: "input",
29+
type: UI.TYPE_INPUT,
3030
items:
3131
[
3232
{

src/ecma-debugger/eventlisteners/evlistenersview.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -122,9 +122,9 @@ cls.EventListenersView = function(id, name, container_class)
122122
this._text_search.set_query_selector(".ev-type");
123123
this._text_search.add_listener("onbeforesearch", this._onbeforesearch.bind(this));
124124
evh.input[cls.EventListenersView.TOKEN_FILTER] = this._oninput.bind(this);
125-
var glob_handl = ActionBroker.get_instance().get_global_handler();
125+
var global_handler = ActionBroker.get_instance().get_global_handler();
126126
var cb = cls.Helpers.shortcut_search_cb.bind(this._text_search);
127-
glob_handl.register_shortcut_listener(cls.EventListenersView.TOKEN_FILTER, cb);
127+
global_handler.register_shortcut_listener(cls.EventListenersView.TOKEN_FILTER, cb);
128128
var contextmenu =
129129
[
130130
{

src/ecma-debugger/eventlisteners/evlistenertemplates.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,9 +38,9 @@
3838
}
3939
var event_types = search_term
4040
? ev_rt.event_types.filter(function(ev_type)
41-
{
42-
return ev_type.type.contains(search_term);
43-
})
41+
{
42+
return ev_type.type.contains(search_term);
43+
})
4444
: ev_rt.event_types;
4545
ret.push(["ul", event_types.map(this._ev_type, this),
4646
"data-rt-id", String(ev_rt.rt_id),

src/ecma-debugger/eventlisteners/selectednodelistenerview.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ cls.SelectedNodeListenersView = function(id, name, container_class)
2929
container.clearAndRender(tmpl);
3030
};
3131

32-
this._on_elemnet_selected = function(msg)
32+
this._on_element_selected = function(msg)
3333
{
3434
this._selected_ele = msg;
3535
this.update();
@@ -39,7 +39,7 @@ cls.SelectedNodeListenersView = function(id, name, container_class)
3939
{
4040
this.init(id, name, container_class);
4141
this._selected_ele = null;
42-
window.messages.add_listener("element-selected", this._on_elemnet_selected.bind(this));
42+
window.messages.add_listener("element-selected", this._on_element_selected.bind(this));
4343
};
4444

4545
this._init(id, name, container_class);

0 commit comments

Comments
 (0)