Commit e4ff6a2f authored by Pedro Eduardo Trujillo's avatar Pedro Eduardo Trujillo
Browse files

Corrige manejo de clicks, roto en 8c019483 

En algunos casos no interesa trabajar con currentTarget de un evento,
sino con target. Los cambios provocaron que no funcionase la selección
desde el árbol o la edición de foto de usuario, por ejemplo.
parent dd8eaee9

public/javascript/app/user/views/UserProfileView.js

+1 −1
Original line number Diff line number Diff line
@@ -359,7 +359,7 @@ define([ 


		_tryToGoToEditImage: function(evt) {



			var node = evt.currentTarget || evt.target,

			var node = evt.target || evt.currentTarget,

				nodeTagName = node.tagName,

				nodeAttribute = node.getAttribute('data-redmic-id');

public/javascript/redmic/modules/base/_ManageClickEvent.js

+4 −2
Original line number Diff line number Diff line
@@ -31,8 +31,10 @@ define([ 
				];



			if (!targetPath.length) {

				var currTargetParent = currTarget.parentElement;

				targetPath.push(currTarget, currTargetParent);

				var eventTarget = event.target || event.currentTarget,

					eventTargetParent = eventTarget.parentElement;



				targetPath.push(eventTarget, eventTargetParent);

			}



			return targets.concat(targetPath);

public/javascript/redmic/modules/tree/Tree.js

+1 −1
Original line number Diff line number Diff line
@@ -222,7 +222,7 @@ define([ 
			var item = evt[0],

				event = evt[2],

				itemId = item[this.idProperty],

				target = event.currentTarget || event.target,

				target = event.target || event.currentTarget,

				evtToEmit = target.checked ? 'SELECT' : 'DESELECT';



			this._emitEvt(evtToEmit, itemId);

tests/unit/redmic/modules/tree/testTree.js

+4 −4
Original line number Diff line number Diff line
@@ -145,7 +145,7 @@ define([ 
					item,

					'nodo',

					{

						currentTarget: {

						target: {

							checked: true

						}

					}
@@ -228,7 +228,7 @@ define([ 
					item,

					'nodo',

					{

						currentTarget: {

						target: {

							checked: false

						}

					}
@@ -611,7 +611,7 @@ define([ 
					item,

					'nodo',

					{

						currentTarget: {

						target: {

							checked: true

						}

					}
@@ -718,7 +718,7 @@ define([ 
					item,

					'nodo',

					{

						currentTarget: {

						target: {

							checked: false

						}

					}