Skip to content

Commit

Permalink
Rename cover image into cover block.
Browse files Browse the repository at this point in the history
  • Loading branch information
jorgefilipecosta committed Oct 17, 2018
1 parent f2a5f1f commit 2da9dac
Show file tree
Hide file tree
Showing 9 changed files with 66 additions and 19 deletions.
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
.wp-block-cover-image {
.wp-block-cover-image,
.wp-block-cover {
.editor-rich-text__tinymce[data-is-empty="true"]::before {
position: inherit;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -60,12 +60,12 @@ const blockAttributes = {
},
};

export const name = 'core/cover-image';
export const name = 'core/cover';

const ALLOWED_MEDIA_TYPES = [ 'image' ];

export const settings = {
title: __( 'Cover Image' ),
title: __( 'Cover' ),

description: __( 'Add a full-width image, and layer text over it — great for headers.' ),

Expand All @@ -81,14 +81,14 @@ export const settings = {
type: 'block',
blocks: [ 'core/heading' ],
transform: ( { content } ) => (
createBlock( 'core/cover-image', { title: content } )
createBlock( 'core/cover', { title: content } )
),
},
{
type: 'block',
blocks: [ 'core/image' ],
transform: ( { caption, url, align, id } ) => (
createBlock( 'core/cover-image', {
createBlock( 'core/cover', {
title: caption,
url,
align,
Expand Down Expand Up @@ -183,7 +183,7 @@ export const settings = {
render={ ( { open } ) => (
<IconButton
className="components-toolbar__control"
label={ __( 'Edit image' ) }
label={ __( 'Edit media' ) }
icon="edit"
onClick={ open }
/>
Expand All @@ -195,7 +195,7 @@ export const settings = {
</BlockControls>
{ !! url && (
<InspectorControls>
<PanelBody title={ __( 'Cover Image Settings' ) }>
<PanelBody title={ __( 'Cover Settings' ) }>
<ToggleControl
label={ __( 'Fixed Background' ) }
checked={ hasParallax }
Expand Down Expand Up @@ -235,7 +235,7 @@ export const settings = {
onChange={ setTitle }
inlineToolbar
/>
) : __( 'Cover Image' );
) : __( 'Cover' );

return (
<Fragment>
Expand All @@ -245,7 +245,7 @@ export const settings = {
className={ className }
labels={ {
title: label,
name: __( 'an image' ),
name: __( 'an image or a video' ),
} }
onSelect={ onSelectImage }
accept="image/*"
Expand All @@ -268,7 +268,7 @@ export const settings = {
{ ( ! RichText.isEmpty( title ) || isSelected ) && (
<RichText
tagName="p"
className="wp-block-cover-image-text"
className="wp-block-cover-text"
placeholder={ __( 'Write title…' ) }
value={ title }
onChange={ setTitle }
Expand Down Expand Up @@ -304,13 +304,50 @@ export const settings = {
return (
<div className={ classes } style={ style }>
{ ! RichText.isEmpty( title ) && (
<RichText.Content tagName="p" className="wp-block-cover-image-text" value={ title } />
<RichText.Content tagName="p" className="wp-block-cover-text" value={ title } />
) }
</div>
);
},

deprecated: [ {
attributes: {
...blockAttributes,
},

supports: {
className: false,
},

save( { attributes } ) {
const { url, title, hasParallax, dimRatio, align, contentAlign, overlayColor, customOverlayColor } = attributes;
const overlayColorClass = getColorClassName( 'background-color', overlayColor );
const style = backgroundImageStyles( url );
if ( ! overlayColorClass ) {
style.backgroundColor = customOverlayColor;
}

const classes = classnames(
'wp-block-cover-image',
dimRatioToClass( dimRatio ),
overlayColorClass,
{
'has-background-dim': dimRatio !== 0,
'has-parallax': hasParallax,
[ `has-${ contentAlign }-content` ]: contentAlign !== 'center',
},
align ? `align${ align }` : null,
);

return (
<div className={ classes } style={ style }>
{ ! RichText.isEmpty( title ) && (
<RichText.Content tagName="p" className="wp-block-cover-image-text" value={ title } />
) }
</div>
);
},
}, {
attributes: {
...blockAttributes,
title: {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
.wp-block-cover-image {
.wp-block-cover-image,
.wp-block-cover {
position: relative;
background-color: $black;
background-size: cover;
Expand All @@ -14,7 +15,8 @@
justify-content: flex-start;

h2,
.wp-block-cover-image-text {
.wp-block-cover-image-text,
.wp-block-cover-text {
margin-left: 0;
text-align: left;
}
Expand All @@ -24,14 +26,16 @@
justify-content: flex-end;

h2,
.wp-block-cover-image-text {
.wp-block-cover-image-text,
.wp-block-cover-text {
margin-right: 0;
text-align: right;
}
}

h2,
.wp-block-cover-image-text {
.wp-block-cover-image-text,
.wp-block-cover-text {
color: $white;
font-size: 2em;
line-height: 1.25;
Expand Down
2 changes: 1 addition & 1 deletion packages/block-library/src/editor.scss
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
@import "./categories/editor.scss";
@import "./code/editor.scss";
@import "./columns/editor.scss";
@import "./cover-image/editor.scss";
@import "./cover/editor.scss";
@import "./embed/editor.scss";
@import "./file/editor.scss";
@import "./classic/editor.scss";
Expand Down
4 changes: 2 additions & 2 deletions packages/block-library/src/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ import * as categories from './categories';
import * as code from './code';
import * as columns from './columns';
import * as column from './columns/column';
import * as coverImage from './cover-image';
import * as cover from './cover';
import * as embed from './embed';
import * as file from './file';
import * as html from './html';
Expand Down Expand Up @@ -69,7 +69,7 @@ export const registerCoreBlocks = () => {
code,
columns,
column,
coverImage,
cover,
embed,
...embed.common,
...embed.others,
Expand Down
2 changes: 1 addition & 1 deletion packages/block-library/src/style.scss
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
@import "./button/style.scss";
@import "./categories/style.scss";
@import "./columns/style.scss";
@import "./cover-image/style.scss";
@import "./cover/style.scss";
@import "./embed/style.scss";
@import "./file/style.scss";
@import "./gallery/style.scss";
Expand Down
5 changes: 5 additions & 0 deletions packages/blocks/src/api/parser.js
Original file line number Diff line number Diff line change
Expand Up @@ -416,6 +416,11 @@ export function createBlockWithFallback( blockNode ) {
// freeform content fallback.
let name = originalName || freeformContentFallbackBlock;

// Convert 'core/cover-image' block in existing content to 'core/cover'.
if ( 'core/cover-image' === name ) {
name = 'core/cover';
}

// Convert 'core/text' blocks in existing content to 'core/paragraph'.
if ( 'core/text' === name || 'core/cover-text' === name ) {
name = 'core/paragraph';
Expand Down

0 comments on commit 2da9dac

Please sign in to comment.