Merge pull request #42 from janniks/allow-fullscreen-video-embed

Add embed allows with fullscreen default

Author: janniks

docs/README.md

@@ -14,6 +14,7 @@ The `NotionRenderer` component offers a few properties
 
 - [`blockMap`](#blockMap) – required
 - [`contentId`](#contentId) – default: `undefined`
+- [`embedAllow`](#embedAllow) – default: `fullscreen`
 - [`fullPage`](#fullPage) – default: `false`
 - [`hideList`](#hideList) – default: `[]`
 - [`mapImageUrl`](#mapImageUrl) – default: `defaultMapImageUrl()`
@@ -31,6 +32,11 @@ A list of blocks by their id that may contain contents and properties.
 If this is `undefined` the _first_ block is rendered.
 _Usually the first block contains the rest of the page._
 
+### `embedAllow`: String
+
+– the [`allow` feature policy](https://developer.mozilla.org/en-US/docs/Web/HTML/Element/iframe#attr-allow) for embedded `<iframe>`s (e.g. YouTube videos).
+The default allows embeds to enter fullscreen.
+
 ### `fullPage`: Boolean
 
 – wether or not the page should contain the cover and header.

src/blocks/helpers/asset.vue

@@ -1,6 +1,6 @@
 <template>
   <div :style="style">
-    <iframe class="notion-image-inset" :src="src" />
+    <iframe class="notion-image-inset" :src="src" :allow="embedAllow" />
   </div>
 </template>
 

src/lib/blockable.js

@@ -3,6 +3,7 @@ import { getTextContent } from "@/lib/utils";
 export const blockProps = {
   blockMap: { type: Object, required: true },
   contentId: { type: String, required: false },
+  embedAllow: { type: String, default: "fullscreen" },
   fullPage: { type: Boolean, default: false },
   hideList: { type: Array, default: () => [] },
   level: { type: Number, default: 0 },
@@ -19,6 +20,7 @@ export const blockComputed = {
     return {
       blockMap: this.blockMap,
       contentId: this.contentId,
+      embedAllow: this.embedAllow,
       fullPage: this.fullPage,
       hideList: this.hideList,
       level: this.level,