From fec8f4e005b87c17b614cc1d4b5d1c8076f7a2bd Mon Sep 17 00:00:00 2001 From: damencho Date: Wed, 26 Oct 2016 17:36:07 -0500 Subject: [PATCH] Fixes an issue with mixing visibility functions for elements. In small video we use UIUtil.setVisibility, so we should not use jquery show/hide for the same element. --- modules/UI/videolayout/LocalVideo.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/modules/UI/videolayout/LocalVideo.js b/modules/UI/videolayout/LocalVideo.js index d806c6efb..021f026b6 100644 --- a/modules/UI/videolayout/LocalVideo.js +++ b/modules/UI/videolayout/LocalVideo.js @@ -114,7 +114,7 @@ LocalVideo.prototype.setDisplayName = function(displayName) { e.preventDefault(); e.stopPropagation(); - $localDisplayName.hide(); + UIUtil.setVisibility($localDisplayName, false); $editDisplayName.show(); $editDisplayName.focus(); $editDisplayName.select(); @@ -122,7 +122,7 @@ LocalVideo.prototype.setDisplayName = function(displayName) { $editDisplayName.one("focusout", function () { self.emitter.emit(UIEvents.NICKNAME_CHANGED, this.value); $editDisplayName.hide(); - $localDisplayName.show(); + UIUtil.setVisibility($localDisplayName, true); }); $editDisplayName.on('keydown', function (e) {