Skip to content

Commit

Permalink
fix(FEC-11887): qna container is not visible on embedded player by de…
Browse files Browse the repository at this point in the history
…fault if the uivar “Kaltura.ForceLayoutRedraw” is set to true (#4257)

**the issue:**
this is a regression caused by SUP-11362.
when uivar Kaltura.ForceLayoutRedraw=true and using qna plugin where onPage=false, the qna container is not visible when the player is loaded.

**the solution:**
executing the responsive action that was added by SUP-11362 only in 2 cases:
1. when there is no qna plugin/disabled.
2. if qna plugin exists and enabled, then only when onPage=true.

Solves FEC-11887
  • Loading branch information
lianbenjamin authored Jan 23, 2022
1 parent f0f71ba commit 017540f
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions modules/EmbedPlayer/resources/mw.EmbedPlayer.js
Original file line number Diff line number Diff line change
Expand Up @@ -1865,9 +1865,13 @@
height: this.getInterface().height() + 1
};
this.updateInterfaceSize(resize);
resize.height = "100%";
resize.width = "100%";
this.updateInterfaceSize(resize);

var qnaPlugin = this.getPluginInstance("qna");
if (!qnaPlugin || qnaPlugin.getConfig("onPage")) {
resize.height = "100%";
resize.width = "100%";
this.updateInterfaceSize(resize);
}
}
this.triggerHelper('widgetLoaded');
}
Expand Down

0 comments on commit 017540f

Please sign in to comment.