Bläddra i källkod

Fixes setting of default display name in remote videos.

j8
yanas 11 år sedan
förälder
incheckning
eec2c487fd
3 ändrade filer med 10 tillägg och 11 borttagningar
  1. 2
    7
      app.js
  2. 1
    2
      index.html
  3. 7
    2
      videolayout.js

+ 2
- 7
app.js Visa fil

@@ -594,14 +594,9 @@ $(document).bind('joined.muc', function (event, jid, info) {
594 594
     // Once we've joined the muc show the toolbar
595 595
     Toolbar.showToolbar();
596 596
 
597
-    var displayName = '';
598 597
     if (info.displayName)
599
-        displayName = info.displayName + ' (me)';
600
-    else
601
-        displayName = "Me";
602
-
603
-    $(document).trigger('displaynamechanged',
604
-                        ['localVideoContainer', displayName]);
598
+        $(document).trigger('displaynamechanged',
599
+                            ['localVideoContainer', info.displayName + ' (me)']);
605 600
 });
606 601
 
607 602
 $(document).bind('entered.muc', function (event, jid, info, pres) {

+ 1
- 2
index.html Visa fil

@@ -58,7 +58,7 @@
58 58
     <link rel="stylesheet" href="css/popup_menu.css?v=2">
59 59
     <link rel="stylesheet" href="css/popover.css?v=1">
60 60
     <link rel="stylesheet" href="css/contact_list.css?v=1">
61
-      <link rel="stylesheet" href="css/welcome_page.css?v=1">
61
+    <link rel="stylesheet" href="css/welcome_page.css?v=1">
62 62
     <!--
63 63
         Link used for inline installation of chrome desktop streaming extension,
64 64
         is updated automatically from the code with the value defined in config.js -->
@@ -277,6 +277,5 @@
277 277
         </div>
278 278
         <a id="downloadlog" onclick='dump(event.target);' data-toggle="popover" data-placement="right" data-content="Download logs" ><i class="fa fa-cloud-download"></i></a>
279 279
     </div>
280
-
281 280
   </body>
282 281
 </html>

+ 7
- 2
videolayout.js Visa fil

@@ -28,6 +28,9 @@ var VideoLayout = (function (my) {
28 28
         var localVideoContainer = document.getElementById('localVideoWrapper');
29 29
         localVideoContainer.appendChild(localVideo);
30 30
 
31
+        // Set default display name.
32
+        setDisplayName('localVideoContainer');
33
+
31 34
         AudioLevels.updateAudioLevelCanvas();
32 35
 
33 36
         var localVideoSelector = $('#' + localVideo.id);
@@ -103,7 +106,6 @@ var VideoLayout = (function (my) {
103 106
         }
104 107
     };
105 108
 
106
-
107 109
     /**
108 110
      * Updates the large video with the given new video source.
109 111
      */
@@ -306,6 +308,9 @@ var VideoLayout = (function (my) {
306 308
             var container
307 309
                 = VideoLayout.addRemoteVideoContainer(peerJid, videoSpanId);
308 310
 
311
+            // Set default display name.
312
+            setDisplayName(videoSpanId);
313
+
309 314
             var nickfield = document.createElement('span');
310 315
             nickfield.className = "nick";
311 316
             nickfield.appendChild(document.createTextNode(resourceJid));
@@ -541,7 +546,7 @@ var VideoLayout = (function (my) {
541 546
                 editableText.className = 'displayname';
542 547
                 editableText.id = 'editDisplayName';
543 548
 
544
-                if (displayName.length) {
549
+                if (displayName && displayName.length) {
545 550
                     editableText.value
546 551
                         = displayName.substring(0, displayName.indexOf(' (me)'));
547 552
                 }

Laddar…
Avbryt
Spara