diff --git a/modules/UI/UI.js b/modules/UI/UI.js index fe61aa21a..fdac5ffd1 100644 --- a/modules/UI/UI.js +++ b/modules/UI/UI.js @@ -1400,13 +1400,13 @@ UI.setMicrophoneButtonEnabled = function (enabled) { UI.showRingOverlay = function () { RingOverlay.show(APP.tokenData.callee, interfaceConfig.DISABLE_RINGING); - FilmStrip.toggleFilmStrip(false); + FilmStrip.toggleFilmStrip(false, false); }; UI.hideRingOverLay = function () { if (!RingOverlay.hide()) return; - FilmStrip.toggleFilmStrip(true); + FilmStrip.toggleFilmStrip(true, false); }; /** diff --git a/modules/UI/videolayout/FilmStrip.js b/modules/UI/videolayout/FilmStrip.js index 762cea584..1fce07002 100644 --- a/modules/UI/videolayout/FilmStrip.js +++ b/modules/UI/videolayout/FilmStrip.js @@ -114,20 +114,24 @@ const FilmStrip = { * of the film strip. If not specified, the visibility will be flipped * (i.e. toggled); otherwise, the visibility will be set to the specified * value. + * @param {Boolean} sendAnalytics - if true analytics event will be send. + * default: true. * * Note: * This method shouldn't be executed directly to hide the filmstrip. * It's important to hide the filmstrip with UI.toggleFilmstrip in order * to correctly resize the video area. */ - toggleFilmStrip(visible) { + toggleFilmStrip(visible, sendAnalytics = true) { let isVisibleDefined = typeof visible === 'boolean'; if (!isVisibleDefined) { visible = this.isFilmStripVisible(); } else if (this.isFilmStripVisible() === visible) { return; } - JitsiMeetJS.analytics.sendEvent('toolbar.filmstrip.toggled'); + if(sendAnalytics) { + JitsiMeetJS.analytics.sendEvent('toolbar.filmstrip.toggled'); + } this.filmStrip.toggleClass("hidden"); if (!visible) {