kaltura / mwEmbed

Kaltura's Cross Platform Video Player ToolKit
http://player.kaltura.com
GNU Affero General Public License v3.0
210 stars 137 forks source link

fix(FEC-11887): qna container is not visible on embedded player by default if the uivar “Kaltura.ForceLayoutRedraw” is set to true #4257

Closed lianbenjamin closed 2 years ago

lianbenjamin commented 2 years ago

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

PlaykitJs-Bot commented 2 years ago

Live Pull Request Urls

PlaykitJs-Bot commented 2 years ago

Updated pull request 4257: 0d1ee83e87557b0683b1d1f291d2929375e4f4c4