Add missing attribute sources; let amp-fit-text attributes pass Kses

ampproject · May 24, 2018 · c3d9915 · c3d9915
1 parent 5a45e80
commit c3d9915
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 3 deletions.
diff --git a/assets/js/amp-editor-blocks.js b/assets/js/amp-editor-blocks.js
@@ -201,15 +201,24 @@ var ampEditorBlocks = ( function() { // eslint-disable-line no-unused-vars
 			};
 			settings.attributes.minFont = {
 				type: 'number',
-				default: component.data.fontSizes.small
+				default: component.data.fontSizes.small,
+				source: 'attribute',
+				selector: 'amp-fit-text',
+				attribute: 'min-font-size'
 			};
 			settings.attributes.maxFont = {
 				type: 'number',
-				default: component.data.fontSizes.larger
+				default: component.data.fontSizes.larger,
+				source: 'attribute',
+				selector: 'amp-fit-text',
+				attribute: 'max-font-size'
 			};
 			settings.attributes.height = {
 				type: 'number',
-				default: 50
+				default: 50,
+				source: 'attribute',
+				selector: 'amp-fit-text',
+				attribute: 'height'
 			};
 		}
 

diff --git a/includes/admin/class-amp-editor-blocks.php b/includes/admin/class-amp-editor-blocks.php
@@ -49,6 +49,7 @@ public function whitelist_block_atts_in_wp_kses_allowed_html( $tags, $context )
 			'amp-jwplayer',
 			'amp-brid-player',
 			'amp-ima-video',
+			'amp-fit-text',
 		);
 
 		foreach ( $amp_blocks as $amp_block ) {
@@ -62,6 +63,7 @@ public function whitelist_block_atts_in_wp_kses_allowed_html( $tags, $context )
 						'layout',
 						'width',
 						'height',
+						'class',
 					),
 					true
 				),