HTML5 Player does not call VPAID 'resizeAd' on fullscreen

[ocundale]

This issue can be seen on the test page here:
coullwebsdk.herokuapp.com/minimal-test.php

• Wait for an overlay to appear, then go fullscreen- Expected functionality is that the overlay should disappear.
  However, because resizeAd is not called, we cannot hide the overlay.

From VPAID spec:
"Following a resize of the ad UI container, the player calls resizeAd to allow the ad to
scale or reposition itself within its display area."

Let me know if you need any further detail. Cheers :)

[mdale]

should the ad disappear or just be resized / re-positioned?

[ocundale]

You just need to call resize ad with the correct adslot dimensions, positioning is handled by the ad, so no need to worry about that.
You need to call the ad with the following params: resizeAd(width, height, viewMode)

• width/height, if specified, should always match the maximum display area allotted for the ad
• viewMode can be one of “normal”, “thumbnail”, or “fullscreen” to indicate the players current viewing mode.

So on fullscreen, the player needs to call resizeAd, this is normally called with parameters like so:
resizeAd(adslot.width, adslot.height, "fullscreen");

The width and height could be null on switching to fullscreen- it is the "fullscreen" string that is important- this allows us to hide the ad.

On exit of fullscreen, you would then call the same function, but with the width and height of the player:
resizeAd(480, 360, "normal");

[amirch1]

Thanks! This will be added to the next player version (2.29)

[mdale]

@amirch1 please let @ocundale know as soon as you have a pull request to try out.

[ocundale]

Good stuff, happy to test once you're ready- Just send over the PR as Michael suggested 👍

[ocundale]

Hi, I have tested this and it seems to work fine in 2.29. However, I should have mentioned that the viewMode also needs to be passed on initAd- at the moment the 'normal' string is hardcoded, but it should pass in 'fullscreen' if the player is actually in fullscreen mode. Does this make sense?! Let me know if you'd like me to create a separate bug.