Settings - Writing: add toggle for Shortcodes. Make it searchable, re…

…move searchable module. Don't activate shortcodes by default. (#12092)
Automattic · Apr 22, 2019 · 6dad7c5 · 6dad7c5
1 parent 4d2875e
commit 6dad7c5
Show file tree

Hide file tree

Showing 4 changed files with 31 additions and 5 deletions.
diff --git a/_inc/client/searchable-modules/index.jsx b/_inc/client/searchable-modules/index.jsx
@@ -45,7 +45,6 @@ export const SearchableModules = withModuleSettingsFormHelpers(
 				'json-api',
 				'latex',
 				'notes',
-				'shortcodes',
 				'shortlinks',
 			];
 

diff --git a/_inc/client/writing/composing.jsx b/_inc/client/writing/composing.jsx
@@ -200,16 +200,18 @@ export class Composing extends React.Component {
 		const foundCopyPost = this.props.isModuleFound( 'copy-post' ),
 			foundAtD = this.props.isModuleFound( 'after-the-deadline' ),
 			foundLatex = this.props.isModuleFound( 'latex' ),
-			foundMarkdown = this.props.isModuleFound( 'markdown' );
+			foundMarkdown = this.props.isModuleFound( 'markdown' ),
+			foundShortcodes = this.props.isModuleFound( 'shortcodes' );
 
-		if ( ! foundCopyPost && ! foundMarkdown && ! foundAtD ) {
+		if ( ! foundCopyPost && ! foundMarkdown && ! foundAtD && ! foundShortcodes ) {
 			return null;
 		}
 
 		const markdown = this.props.module( 'markdown' ),
 			latex = this.props.module( 'latex' ),
 			atd = this.props.module( 'after-the-deadline' ),
 			copyPost = this.props.module( 'copy-post' ),
+			shortcodes = this.props.module( 'shortcodes' ),
 			unavailableInDevMode = this.props.isUnavailableInDevMode( 'after-the-deadline' ),
 			copyPostSettings = (
 				<SettingsGroup
@@ -289,6 +291,29 @@ export class Composing extends React.Component {
 					</FormFieldset>
 				</SettingsGroup>
 			),
+			shortcodeSettings = (
+				<SettingsGroup
+					module={ shortcodes }
+					support={ {
+						text: shortcodes.description,
+						link: 'https://jetpack.com/support/shortcode-embeds/',
+					} }
+				>
+					<FormFieldset>
+						<ModuleToggle
+							slug="shortcodes"
+							activated={ !! this.props.getOptionValue( 'shortcodes' ) }
+							toggling={ this.props.isSavingAnyOption( [ 'shortcodes' ] ) }
+							disabled={ this.props.isSavingAnyOption( [ 'shortcodes' ] ) }
+							toggleModule={ this.props.toggleModuleNow }
+						>
+							<span className="jp-form-toggle-explanation">
+								{ __( 'Compose using shortcodes to embed media from popular sites' ) }
+							</span>
+						</ModuleToggle>
+					</FormFieldset>
+				</SettingsGroup>
+			),
 			atdSettings = (
 				<FoldableCard
 					onOpen={ this.trackOpenCard }
@@ -327,6 +352,7 @@ export class Composing extends React.Component {
 				{ foundCopyPost && copyPostSettings }
 				{ foundMarkdown && markdownSettings }
 				{ foundLatex && latexSettings }
+				{ foundShortcodes && shortcodeSettings }
 				{ foundAtD && atdSettings }
 			</SettingsCard>
 		);

diff --git a/_inc/client/writing/index.jsx b/_inc/client/writing/index.jsx
@@ -46,6 +46,7 @@ export class Writing extends React.Component {
 			'copy-post',
 			'masterbar',
 			'markdown',
+			'shortcodes',
 			'after-the-deadline',
 			'custom-content-types',
 			'post-by-email',

diff --git a/modules/shortcodes.php b/modules/shortcodes.php
@@ -2,12 +2,12 @@
 
 /**
  * Module Name: Shortcode Embeds
- * Module Description: Embed media from popular sites without any coding.
+ * Module Description: Shortcodes are WordPress-specific markup that let you add media from popular sites. This feature is no longer necessary as the editor now handles media embeds rather gracefully.
  * Sort Order: 3
  * First Introduced: 1.1
  * Major Changes In: 1.2
  * Requires Connection: No
- * Auto Activate: Yes
+ * Auto Activate: No
  * Module Tags: Photos and Videos, Social, Writing, Appearance
  * Feature: Writing
  * Additional Search Queries: shortcodes, shortcode, embeds, media, bandcamp, dailymotion, facebook, flickr, google calendars, google maps, google+, polldaddy, recipe, recipes, scribd, slideshare, slideshow, slideshows, soundcloud, ted, twitter, vimeo, vine, youtube