Sfoglia il codice sorgente

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.
master
damencho 9 anni fa
parent
commit
fec8f4e005
1 ha cambiato i file con 2 aggiunte e 2 eliminazioni
  1. 2
    2
      modules/UI/videolayout/LocalVideo.js

+ 2
- 2
modules/UI/videolayout/LocalVideo.js Vedi File

114
 
114
 
115
                 e.preventDefault();
115
                 e.preventDefault();
116
                 e.stopPropagation();
116
                 e.stopPropagation();
117
-                $localDisplayName.hide();
117
+                UIUtil.setVisibility($localDisplayName, false);
118
                 $editDisplayName.show();
118
                 $editDisplayName.show();
119
                 $editDisplayName.focus();
119
                 $editDisplayName.focus();
120
                 $editDisplayName.select();
120
                 $editDisplayName.select();
122
                 $editDisplayName.one("focusout", function () {
122
                 $editDisplayName.one("focusout", function () {
123
                     self.emitter.emit(UIEvents.NICKNAME_CHANGED, this.value);
123
                     self.emitter.emit(UIEvents.NICKNAME_CHANGED, this.value);
124
                     $editDisplayName.hide();
124
                     $editDisplayName.hide();
125
-                    $localDisplayName.show();
125
+                    UIUtil.setVisibility($localDisplayName, true);
126
                 });
126
                 });
127
 
127
 
128
                 $editDisplayName.on('keydown', function (e) {
128
                 $editDisplayName.on('keydown', function (e) {

Loading…
Annulla
Salva