Revert #2628

WordPress · Jan 18, 2018 · c4e7bee · c4e7bee
1 parent 6c256dd
commit c4e7bee
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 59 deletions.
diff --git a/blocks/editable/format-toolbar/index.js b/blocks/editable/format-toolbar/index.js
@@ -12,7 +12,6 @@ import { keycodes } from '@wordpress/utils';
 import './style.scss';
 import UrlInput from '../../url-input';
 import { filterURLForDisplay } from '../../../editor/utils/url';
-import ToggleControl from '../../inspector-controls/toggle-control';
 
 const { ESCAPE, LEFT, RIGHT, UP, DOWN } = keycodes;
 
@@ -48,11 +47,10 @@ const stopKeyPropagation = ( event ) => event.stopPropagation();
 class FormatToolbar extends Component {
 	constructor() {
 		super( ...arguments );
+
 		this.state = {
 			isAddingLink: false,
 			isEditingLink: false,
-			settingsVisible: false,
-			opensInNewWindow: false,
 			newLinkValue: '',
 		};
 
@@ -62,8 +60,6 @@ class FormatToolbar extends Component {
 		this.submitLink = this.submitLink.bind( this );
 		this.onKeyDown = this.onKeyDown.bind( this );
 		this.onChangeLinkValue = this.onChangeLinkValue.bind( this );
-		this.toggleLinkSettingsVisibility = this.toggleLinkSettingsVisibility.bind( this );
-		this.setLinkTarget = this.setLinkTarget.bind( this );
 	}
 
 	onKeyDown( event ) {
@@ -83,8 +79,6 @@ class FormatToolbar extends Component {
 			this.setState( {
 				isAddingLink: false,
 				isEditingLink: false,
-				settingsVisible: false,
-				opensInNewWindow: !! nextProps.formats.link && !! nextProps.formats.link.target,
 				newLinkValue: '',
 			} );
 		}
@@ -102,16 +96,6 @@ class FormatToolbar extends Component {
 		};
 	}
 
-	toggleLinkSettingsVisibility() {
-		this.setState( ( state ) => ( { settingsVisible: ! state.settingsVisible } ) );
-	}
-
-	setLinkTarget( event ) {
-		const opensInNewWindow = event.target.checked;
-		this.setState( { opensInNewWindow } );
-		this.props.onChange( { link: { value: this.props.formats.link.value, target: opensInNewWindow ? '_blank' : '' } } );
-	}
-
 	addLink() {
 		this.setState( { isEditingLink: false, isAddingLink: true, newLinkValue: '' } );
 	}
@@ -128,7 +112,7 @@ class FormatToolbar extends Component {
 
 	submitLink( event ) {
 		event.preventDefault();
-		this.props.onChange( { link: { value: this.state.newLinkValue, target: this.state.opensInNewWindow ? '_blank' : '' } } );
+		this.props.onChange( { link: { value: this.state.newLinkValue } } );
 		if ( this.state.isAddingLink ) {
 			this.props.speak( __( 'Link added.' ), 'assertive' );
 		}
@@ -140,7 +124,7 @@ class FormatToolbar extends Component {
 
 	render() {
 		const { formats, focusPosition, enabledControls = DEFAULT_CONTROLS, customControls = [] } = this.props;
-		const { isAddingLink, isEditingLink, newLinkValue, settingsVisible, opensInNewWindow } = this.state;
+		const { isAddingLink, isEditingLink, newLinkValue } = this.state;
 		const linkStyle = focusPosition ?
 			{ position: 'absolute', ...focusPosition } :
 			null;
@@ -156,15 +140,6 @@ class FormatToolbar extends Component {
 				};
 			} );
 
-		const linkSettings = settingsVisible && (
-			<div className="blocks-format-toolbar__link-modal-line blocks-format-toolbar__link-settings">
-				<ToggleControl
-					label={ __( 'Open in new window' ) }
-					checked={ opensInNewWindow }
-					onChange={ this.setLinkTarget } />
-			</div>
-		);
-
 		return (
 			<div className="blocks-format-toolbar">
 				<Toolbar controls={ toolbarControls } />
@@ -183,13 +158,7 @@ class FormatToolbar extends Component {
 								<UrlInput value={ newLinkValue } onChange={ this.onChangeLinkValue } />
 								<IconButton icon="editor-break" label={ __( 'Apply' ) } type="submit" />
 								<IconButton icon="editor-unlink" label={ __( 'Remove link' ) } onClick={ this.dropLink } />
-								<IconButton
-									icon="admin-generic"
-									label={ __( 'Link Settings' ) }
-									onClick={ this.toggleLinkSettingsVisibility }
-									aria-expanded={ settingsVisible } />
 							</div>
-							{ linkSettings }
 						</form>
 					</Fill>
 					/* eslint-enable jsx-a11y/no-noninteractive-element-interactions */
@@ -214,13 +183,7 @@ class FormatToolbar extends Component {
 								</a>
 								<IconButton icon="edit" label={ __( 'Edit' ) } onClick={ this.editLink } />
 								<IconButton icon="editor-unlink" label={ __( 'Remove link' ) } onClick={ this.dropLink } />
-								<IconButton
-									icon="admin-generic"
-									label={ __( 'Link Settings' ) }
-									onClick={ this.toggleLinkSettingsVisibility }
-									aria-expanded={ settingsVisible } />
 							</div>
-							{ linkSettings }
 						</div>
 					</Fill>
 					/* eslint-enable jsx-a11y/no-static-element-interactions */

diff --git a/blocks/editable/format-toolbar/style.scss b/blocks/editable/format-toolbar/style.scss
@@ -7,17 +7,13 @@
 	box-shadow: 0px 3px 20px rgba( 18, 24, 30, .1 ), 0px 1px 3px rgba( 18, 24, 30, .1 );
 	border: 1px solid #e0e5e9;
 	background: #fff;
-	width: 305px;
+	width: 300px;
 	display: flex;
 	flex-direction: column;
 	font-family: $default-font;
 	font-size: $default-font-size;
 	line-height: $default-line-height;
 	z-index: z-index( '.blocks-format-toolbar__link-modal' );
-
-	.blocks-url-input {
-		width: auto;
-	}
 }
 
 .blocks-format-toolbar__link-modal-line {
@@ -46,15 +42,3 @@
 		@include long-content-fade( $size: 40% );
 	}
 }
-
-.blocks-format-toolbar__link-settings {
-	padding: 7px 8px;
-	border-top: 1px solid $light-gray-500;
-	padding-top: 8px; // add 1px for the border
-
-	.blocks-base-control {
-		margin: 0;
-		flex-grow: 1;
-		flex-shrink: 1;
-	}
-}
diff --git a/blocks/editable/index.js b/blocks/editable/index.js
@@ -63,7 +63,7 @@ function getFormatProperties( formatName, parents ) {
 	switch ( formatName ) {
 		case 'link' : {
 			const anchor = find( parents, node => node.nodeName.toLowerCase() === 'a' );
-			return !! anchor ? { value: anchor.getAttribute( 'href' ) || '', target: anchor.getAttribute( 'target' ) || '', node: anchor } : {};
+			return !! anchor ? { value: anchor.getAttribute( 'href' ) || '', node: anchor } : {};
 		}
 		default:
 			return {};
@@ -804,7 +804,7 @@ export default class Editable extends Component {
 					if ( ! anchor ) {
 						this.removeFormat( 'link' );
 					}
-					this.applyFormat( 'link', { href: formatValue.value, target: formatValue.target }, anchor );
+					this.applyFormat( 'link', { href: formatValue.value }, anchor );
 				} else {
 					this.editor.execCommand( 'Unlink' );
 				}