Skip to content

Commit

Permalink
Align property names
Browse files Browse the repository at this point in the history
  • Loading branch information
David Aurelio committed Mar 1, 2013
1 parent 70d7133 commit 23a7412
Show file tree
Hide file tree
Showing 5 changed files with 26 additions and 26 deletions.
2 changes: 1 addition & 1 deletion src/bootstrapper/player.js
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ function(EventEmitter, RendererController, AssetController, tools, URI, version)
var context = new this.RunnerContext(this.runnerUrl, doc, options.baseUrl);
var renderer = new this.Renderer(node, width, height, {
allowEventDefaults: options.allowEventDefaults,
elementsUnderPointer: options.elementsUnderPointer,
objectsUnderPointer: options.objectsUnderPointer,
fpsLog: options.fpsLog,
disableContextMenu: options.disableContextMenu
});
Expand Down
2 changes: 1 addition & 1 deletion src/renderer/renderer_controller.js
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ function(tools, EventEmitter, URI) {
event: event,
targetId: targetId,
relatedTargetId: relatedTargetId,
elementsUnderPointerIds: underPointerIds
objectsUnderPointerIds: underPointerIds
});
});

Expand Down
32 changes: 16 additions & 16 deletions src/renderer/svg/svg_event_handlers.js
Original file line number Diff line number Diff line change
Expand Up @@ -81,14 +81,14 @@ define([
* @param {PointerEvent} event The pointer event to dispatch
* @param {number} targetId The bonsai id of the event target
* @param {number} [relatedTargetId] The bonsai id of the related target, if any
* @param {Array} [elementsUnderPointerIds] An array of element ids under the mouse pointer
* @param {Array} [objectsUnderPointerIds] An array of element ids under the mouse pointer
*/
function emitMouseEvent(emitter, event, targetId, relatedTargetId, elementsUnderPointerIds) {
function emitMouseEvent(emitter, event, targetId, relatedTargetId, objectsUnderPointerIds) {
emitter.emit('userevent', event, targetId, relatedTargetId);
if (!TOUCH_SUPPORT) {
// If we're on a non-touch platform (e.g. regular desktop)
// then fire the mutli: event so we get cross-platform support:
emitter.emit('userevent', event.clone('multi:' + event.type), targetId, relatedTargetId, elementsUnderPointerIds);
emitter.emit('userevent', event.clone('multi:' + event.type), targetId, relatedTargetId, objectsUnderPointerIds);
}
}

Expand All @@ -97,14 +97,14 @@ define([
* @param {PointerEvent} event The pointer event to dispatch
* @param {number} targetId The bonsai id of the event target
* @param {boolean} isMultiTouch Whether the touch is part of a multitouch gesture
* @param {Array} [elementsUnderPointerIds] An array of element ids under the finger tip
* @param {Array} [objectsUnderPointerIds] An array of element ids under the finger tip
*/
function emitTouchEvent(emitter, event, targetId, isMultiTouch, elementsUnderPointerIds) {
function emitTouchEvent(emitter, event, targetId, isMultiTouch, objectsUnderPointerIds) {
var type = event.type;
event.type = 'multi:' + type;
emitter.emit('userevent', event, targetId);
if (!isMultiTouch) {
emitter.emit('userevent', event.clone(type), targetId, null, elementsUnderPointerIds);
emitter.emit('userevent', event.clone(type), targetId, null, objectsUnderPointerIds);
}
}

Expand Down Expand Up @@ -197,9 +197,9 @@ define([
var type = pointerEvent.type, x = pointerEvent.x, y = pointerEvent.y;
if (!type) { return; }

var elementIdsUnderPointer;
if (this.elementsUnderPointer) {
elementIdsUnderPointer = this.getElementIdsUnderPointer(x, y);
var objectUnderPointerIds;
if (this.objectsUnderPointer) {
objectUnderPointerIds = this.getElementIdsUnderPointer(x, y);
}

if (type === 'pointerdown') {
Expand All @@ -214,14 +214,14 @@ define([
var dragEvent = pointerEvent.clone('drag');
dragEvent.diffX = x - this._mouseDragStartX;
dragEvent.diffY = y - this._mouseDragStartY;
emitMouseEvent(this, dragEvent, dragId, null, elementIdsUnderPointer);
emitMouseEvent(this, dragEvent, dragId, null, objectUnderPointerIds);
}
} else if (type === 'pointerup') {
this._mouseDragId = this._mouseDragStartX = this._mouseDragStartY = undefined;
}
this._mouseMoveLastX = x;
this._mouseMoveLastY = y;
emitMouseEvent(this, pointerEvent, targetId, relatedTargetId, elementIdsUnderPointer);
emitMouseEvent(this, pointerEvent, targetId, relatedTargetId, objectUnderPointerIds);
},

/**
Expand All @@ -236,9 +236,9 @@ define([

var x = pointerEvent.x, y = pointerEvent.y;

var elementIdsUnderPointer;
if (this.elementsUnderPointer) {
elementIdsUnderPointer = this.getElementIdsUnderPointer(x, y);
var objectUnderPointerIds;
if (this.objectsUnderPointer) {
objectUnderPointerIds = this.getElementIdsUnderPointer(x, y);
}

var isMultiTouch = this._isMultiTouch;
Expand All @@ -257,12 +257,12 @@ define([
pointerEvent.diffY = y - touchData.dragStartY;
pointerEvent.deltaX = x - touchData.lastX;
pointerEvent.deltaY = y - touchData.lastY;
emitTouchEvent(this, pointerEvent.clone('drag'), targetId, isMultiTouch, elementIdsUnderPointer);
emitTouchEvent(this, pointerEvent.clone('drag'), targetId, isMultiTouch, objectUnderPointerIds);
}
touchData.lastX = x;
touchData.lastY = y;
}
emitTouchEvent(this, pointerEvent, targetId, isMultiTouch, elementIdsUnderPointer);
emitTouchEvent(this, pointerEvent, targetId, isMultiTouch, objectUnderPointerIds);
},

getElementIdsUnderPointer: function(x, y) {
Expand Down
10 changes: 5 additions & 5 deletions src/runner/stage.js
Original file line number Diff line number Diff line change
Expand Up @@ -147,11 +147,11 @@ define([
event.relatedTarget = displayObjectsRegistry[relatedTargetId] || this;
}

var elementsUnderPointerIds = data.elementsUnderPointerIds;
if (elementsUnderPointerIds) {
var elementsUnderPointer = event.underPointer = [];
for (var i = 0, elementId; (elementId = elementsUnderPointerIds[i]); i += 1) {
elementsUnderPointer[i] = displayObjectsRegistry[elementId];
var objectsUnderPointerIds = data.objectsUnderPointerIds;
if (objectsUnderPointerIds) {
var objectsUnderPointer = event.underPointer = [];
for (var i = 0, elementId; (elementId = objectsUnderPointerIds[i]); i += 1) {
objectsUnderPointer[i] = displayObjectsRegistry[elementId];
}
}

Expand Down
6 changes: 3 additions & 3 deletions test/player-spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -134,13 +134,13 @@ define([
expect(funcSetup(createMockNode())).toBeInstanceOf(player.RendererController);
});

it('passes node, width, height, allowEventDefaults, elementsUnderPointer, and fpsLog arguments to the renderer', function () {
var node = createMockNode(), width = 162, height = 100, options = {allowEventDefaults: true, elementsUnderPointer: true, fpsLog: true};
it('passes node, width, height, allowEventDefaults, objectsUnderPointer, and fpsLog arguments to the renderer', function () {
var node = createMockNode(), width = 162, height = 100, options = {allowEventDefaults: true, objectsUnderPointer: true, fpsLog: true};
funcSetup(node, width, height, options);

expect(MockRendererConstructor).toHaveBeenCalledWith(node, width, height, {
allowEventDefaults: options.allowEventDefaults,
elementsUnderPointer: true,
objectsUnderPointer: true,
fpsLog: options.fpsLog
});
});
Expand Down

0 comments on commit 23a7412

Please sign in to comment.