Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Gutenberg] Add amp-timeago block #1168

Merged
merged 18 commits into from
May 30, 2018
Merged

[Gutenberg] Add amp-timeago block #1168

Show file tree
Hide file tree
Changes from 14 commits
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
b051230
Restore amp-timeago block.
miina May 23, 2018
e018851
Fix eslint.
miina May 23, 2018
75e52a5
Merge branch 'develop' of https://github.com/Automattic/amp-wp into a…
westonruter May 23, 2018
9529d2c
Add class to list of attributes always allowed by Kses in AMP components
westonruter May 23, 2018
d8a373f
Merge develop & resolve conflicts.
miina May 28, 2018
470c809
Get param values from saved attributes.
miina May 28, 2018
f1892eb
Merge branch 'develop' of https://github.com/Automattic/amp-wp into a…
westonruter May 30, 2018
f65439a
Fix order of amp-editor-blocks inline scripts
westonruter May 30, 2018
e27811c
Merge develop & resolve conflicts.
miina May 30, 2018
b57265a
Fix domain not found error.
miina May 30, 2018
384fc32
Resolve conflicts.
miina May 30, 2018
b396eb3
Fix layout attribute.
miina May 30, 2018
0bd1a9f
Fix typo + saving attribute.
miina May 30, 2018
e35039e
Use layout instead of ampLayout as the attribute within amp-timeago b…
miina May 30, 2018
11fc94b
Refactor code to decrease repetition.
miina May 30, 2018
9f2343f
Add missing React component keys and fix typo in isDismissible
westonruter May 30, 2018
dbc4ed8
Use ampLayout instead of layout for attribute name in amp-timeago
westonruter May 30, 2018
f4464ca
Update layout to ampLayout within all AMP blocks.
miina May 30, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
37 changes: 29 additions & 8 deletions blocks/amp-brid-player/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,34 +29,55 @@ export default registerBlockType(

attributes: {
autoPlay: {
default: false
type: 'boolean'
},
dataPartner: {
type: 'number'
type: 'number',
source: 'attribute',
selector: 'amp-brid-player',
attribute: 'data-partner'
},
dataPlayer: {
type: 'number'
type: 'number',
source: 'attribute',
selector: 'amp-brid-player',
attribute: 'data-player'
},
dataVideo: {
type: 'number'
type: 'number',
source: 'attribute',
selector: 'amp-brid-player',
attribute: 'data-video'
},
dataPlaylist: {
type: 'number'
type: 'number',
source: 'attribute',
selector: 'amp-brid-player',
attribute: 'data-playlist'
},
dataOutstream: {
type: 'number'
type: 'number',
source: 'attribute',
selector: 'amp-brid-player',
attribute: 'data-outstream'
},
layout: {
type: 'string',
default: 'responsive'
default: 'responsive',
source: 'attribute',
selector: 'amp-brid-player',
attribute: 'layout'
},
width: {
type: 'number',
default: 600
},
height: {
type: 'number',
default: 400
default: 400,
source: 'attribute',
selector: 'amp-brid-player',
attribute: 'height'
}
},

Expand Down
35 changes: 28 additions & 7 deletions blocks/amp-ima-video/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,28 +30,49 @@ export default registerBlockType(
// @todo Perhaps later add subtitles option and additional source options?
attributes: {
dataDelayAdRequest: {
default: false
default: false,
source: 'attribute',
selector: 'amp-ima-video',
attribute: 'data-delay-ad-request'
},
dataTag: {
type: 'string'
type: 'string',
source: 'attribute',
selector: 'amp-ima-video',
attribute: 'data-tag'
},
dataSrc: {
type: 'string'
type: 'string',
source: 'attribute',
selector: 'amp-ima-video',
attribute: 'data-src'
},
dataPoster: {
type: 'string'
type: 'string',
source: 'attribute',
selector: 'amp-ima-video',
attribute: 'data-poster'
},
layout: {
type: 'string',
default: 'responsive'
default: 'responsive',
source: 'attribute',
selector: 'amp-ima-video',
attribute: 'layout'
},
width: {
type: 'number',
default: 600
default: 600,
source: 'attribute',
selector: 'amp-ima-video',
attribute: 'width'
},
height: {
type: 'number',
default: 400
default: 400,
source: 'attribute',
selector: 'amp-ima-video',
attribute: 'height'
}
},

Expand Down
33 changes: 26 additions & 7 deletions blocks/amp-jwplayer/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,25 +28,43 @@ export default registerBlockType(

attributes: {
dataPlayerId: {
type: 'string'
type: 'string',
source: 'attribute',
selector: 'amp-jwplayer',
attribute: 'data-player-id'
},
dataMediaId: {
type: 'string'
type: 'string',
source: 'attribute',
selector: 'amp-jwplayer',
attribute: 'data-media-id'
},
dataPlaylistId: {
type: 'string'
type: 'string',
source: 'attribute',
selector: 'amp-jwplayer',
attribute: 'data-playlist-id'
},
layout: {
type: 'string',
default: 'responsive'
default: 'responsive',
source: 'attribute',
selector: 'amp-jwplayer',
attribute: 'layout'
},
width: {
type: 'number',
default: 600
default: 600,
source: 'attribute',
selector: 'amp-jwplayer',
attribute: 'width'
},
height: {
type: 'number',
default: 400
default: 400,
source: 'attribute',
selector: 'amp-jwplayer',
attribute: 'height'
}
},

Expand Down Expand Up @@ -142,7 +160,8 @@ export default registerBlockType(
}
if ( attributes.dataPlaylistId ) {
jwProps[ 'data-playlist-id' ] = attributes.dataPlaylistId;
} else if ( attributes.dataMediaId ) {
}
if ( attributes.dataMediaId ) {
jwProps[ 'data-media-id' ] = attributes.dataMediaId;
}
return (
Expand Down
5 changes: 4 additions & 1 deletion blocks/amp-mathml/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,10 @@ export default registerBlockType(

attributes: {
dataFormula: {
type: 'string'
type: 'string',
source: 'attribute',
selector: 'amp-mathml',
attribute: 'data-formula'
}
},

Expand Down
35 changes: 28 additions & 7 deletions blocks/amp-o2-player/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,32 +30,53 @@ export default registerBlockType(
// @todo Add other useful macro toggles, e.g. showing relevant content.
attributes: {
dataPid: {
type: 'string'
type: 'string',
source: 'attribute',
selector: 'amp-o2-player',
attribute: 'data-pid'
},
dataVid: {
type: 'string'
type: 'string',
source: 'attribute',
selector: 'amp-o2-player',
attribute: 'data-vid'
},
dataBcid: {
type: 'string'
type: 'string',
source: 'attribute',
selector: 'amp-o2-player',
attribute: 'data-bcid'
},
dataBid: {
type: 'string'
type: 'string',
source: 'attribute',
selector: 'amp-o2-player',
attribute: 'data-bid'
},
autoPlay: {
type: 'boolean',
default: false
},
layout: {
type: 'string',
default: 'responsive'
default: 'responsive',
source: 'attribute',
selector: 'amp-o2-player',
attribute: 'layout'
},
width: {
type: 'number',
default: 600
default: 600,
source: 'attribute',
selector: 'amp-o2-player',
attribute: 'width'
},
height: {
type: 'number',
default: 400
default: 400,
source: 'attribute',
selector: 'amp-o2-player',
attribute: 'height'
}
},

Expand Down
35 changes: 28 additions & 7 deletions blocks/amp-ooyala-player/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,29 +30,50 @@ export default registerBlockType(
// @todo Add data-config attribute?
attributes: {
dataEmbedCode: {
type: 'string'
type: 'string',
source: 'attribute',
selector: 'amp-ooyala-player',
attribute: 'data-embedcode'
},
dataPlayerId: {
type: 'string'
type: 'string',
source: 'attribute',
selector: 'amp-ooyala-player',
attribute: 'data-playerid'
},
dataPcode: {
type: 'string'
type: 'string',
source: 'attribute',
selector: 'amp-ooyala-player',
attribute: 'data-pcode'
},
dataPlayerVersion: {
type: 'string',
default: 'v3'
default: 'v3',
source: 'attribute',
selector: 'amp-ooyala-player',
attribute: 'data-playerversion'
},
layout: {
type: 'string',
default: 'fixed'
default: 'fixed',
source: 'attribute',
selector: 'amp-ooyala-player',
attribute: 'layout'
},
width: {
type: 'number',
default: 600
default: 600,
source: 'attribute',
selector: 'amp-ooyala-player',
attribute: 'width'
},
height: {
type: 'number',
default: 400
default: 400,
source: 'attribute',
selector: 'amp-ooyala-player',
attribute: 'height'
}
},

Expand Down
20 changes: 16 additions & 4 deletions blocks/amp-reach-player/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,19 +29,31 @@ export default registerBlockType(

attributes: {
dataEmbedId: {
type: 'string'
type: 'string',
source: 'attribute',
selector: 'amp-reach-player',
attribute: 'data-embed-id'
},
layout: {
type: 'string',
default: 'fixed-height'
default: 'fixed-height',
source: 'attribute',
selector: 'amp-reach-player',
attribute: 'layout'
},
width: {
type: 'number',
default: 600
default: 600,
source: 'attribute',
selector: 'amp-reach-player',
attribute: 'width'
},
height: {
type: 'number',
default: 400
default: 400,
source: 'attribute',
selector: 'amp-reach-player',
attribute: 'height'
}
},

Expand Down
Loading