Hide fullscreen button for video/DQ of all Columns
Description
Acceptance criteria
It was not intentional to limit hiding the fullscreen button in Interactive Book only (see background).
Code is fine.
Background
The fullscreen button on Column subcontent can create a weird user experience. While sending "Video" to fullscreen seems to work on Chrome, iOS is known to cause trouble and only sending some subcontent to fullscreen may be confusing at times. Sending "Drag Question" to fullscreen even crashes.
There are two extra classes inside Column that are specific to Interactive Book only. It feels like a hotfix. And if the change that's done was limited to Interactive Book intentionally, one would expect those extra CSS classes in the code of Interactive Book instead, not here in Column.
Suggestion
The fullscreen button of "Video" instances and "Drag Question" instances will be hidden for all "Column" instances, not only for Interactive Book.
Acceptance criteria
It was not intentional to limit hiding the fullscreen button in Interactive Book only (see background).
Code is fine.
Background
The fullscreen button on Column subcontent can create a weird user experience. While sending "Video" to fullscreen seems to work on Chrome, iOS is known to cause trouble and only sending some subcontent to fullscreen may be confusing at times. Sending "Drag Question" to fullscreen even crashes.
There are two extra classes inside Column that are specific to Interactive Book only. It feels like a hotfix. And if the change that's done was limited to Interactive Book intentionally, one would expect those extra CSS classes in the code of Interactive Book instead, not here in Column.
Suggestion
The fullscreen button of "Video" instances and "Drag Question" instances will be hidden for all "Column" instances, not only for Interactive Book.
Pull request: