From fe8a15cb6ce4c550d08732e6eec0ce266d181e25 Mon Sep 17 00:00:00 2001
From: Daniel Richards <daniel.richards@automattic.com>
Date: Thu, 2 Aug 2018 17:44:53 +0800
Subject: [PATCH] Rename class name to reflect component is now in edit-post,
 and reorder shortcuts

---
 .../keyboard-shortcut-help-modal/config.js    |  8 +++----
 .../keyboard-shortcut-help-modal/index.js     | 20 ++++++++---------
 .../keyboard-shortcut-help-modal/style.scss   |  2 +-
 .../test/__snapshots__/index.js.snap          | 22 ++++++++++++-------
 4 files changed, 29 insertions(+), 23 deletions(-)

diff --git a/edit-post/components/keyboard-shortcut-help-modal/config.js b/edit-post/components/keyboard-shortcut-help-modal/config.js
index 832a98c7bac28b..05e7633ccf33f1 100644
--- a/edit-post/components/keyboard-shortcut-help-modal/config.js
+++ b/edit-post/components/keyboard-shortcut-help-modal/config.js
@@ -15,10 +15,6 @@ const globalShortcuts = {
 			key: displayShortcut.primary( 's' ),
 			description: __( 'Save your changes.' ),
 		},
-		{
-			key: displayShortcut.primaryShift( ',' ),
-			description: __( 'Show or hide the settings sidebar.' ),
-		},
 		{
 			key: displayShortcut.primary( 'z' ),
 			description: __( 'Undo your last changes.' ),
@@ -27,6 +23,10 @@ const globalShortcuts = {
 			key: displayShortcut.primaryShift( 'z' ),
 			description: __( 'Redo your last undo.' ),
 		},
+		{
+			key: displayShortcut.primaryShift( ',' ),
+			description: __( 'Show or hide the settings sidebar.' ),
+		},
 		{
 			key: displayShortcut.secondary( 'm' ),
 			description: __( 'Switch between Visual Editor and Code Editor.' ),
diff --git a/edit-post/components/keyboard-shortcut-help-modal/index.js b/edit-post/components/keyboard-shortcut-help-modal/index.js
index ab2d444c0b3c71..e7f7cf9290000d 100644
--- a/edit-post/components/keyboard-shortcut-help-modal/index.js
+++ b/edit-post/components/keyboard-shortcut-help-modal/index.js
@@ -25,7 +25,7 @@ const splitShortcutKey = ( shortcutKey ) => {
 				return (
 					<kbd
 						key={ index }
-						className="editor-keyboard-shortcut-help__shortcut-key"
+						className="edit-post-keyboard-shortcut-help__shortcut-key"
 					>
 						{ character }
 					</kbd>
@@ -41,18 +41,18 @@ const splitShortcutKey = ( shortcutKey ) => {
 };
 
 const ShortcutList = ( { shortcuts } ) => (
-	<dl className="editor-keyboard-shortcut-help__shortcut-list">
+	<dl className="edit-post-keyboard-shortcut-help__shortcut-list">
 		{ shortcuts.map( ( { key, description }, index ) => (
 			<div
-				className="editor-keyboard-shortcut-help__shortcut"
+				className="edit-post-keyboard-shortcut-help__shortcut"
 				key={ index }
 			>
-				<dt className="editor-keyboard-shortcut-help__shortcut-term">
-					<kbd className="editor-keyboard-shortcut-help__shortcut-key-combination">
+				<dt className="edit-post-keyboard-shortcut-help__shortcut-term">
+					<kbd className="edit-post-keyboard-shortcut-help__shortcut-key-combination">
 						{ splitShortcutKey( key ) }
 					</kbd>
 				</dt>
-				<dd className="editor-keyboard-shortcut-help__shortcut-description">
+				<dd className="edit-post-keyboard-shortcut-help__shortcut-description">
 					{ description }
 				</dd>
 			</div>
@@ -61,8 +61,8 @@ const ShortcutList = ( { shortcuts } ) => (
 );
 
 const ShortcutSection = ( { title, shortcuts } ) => (
-	<section className="editor-keyboard-shortcut-help__section">
-		<h2 className="editor-keyboard-shortcut-help__section-title">
+	<section className="edit-post-keyboard-shortcut-help__section">
+		<h2 className="edit-post-keyboard-shortcut-help__section-title">
 			{ title }
 		</h2>
 		<ShortcutList shortcuts={ shortcuts } />
@@ -71,7 +71,7 @@ const ShortcutSection = ( { title, shortcuts } ) => (
 
 export function KeyboardShortcutHelpModal( { isModalActive, toggleModal } ) {
 	const title = (
-		<span className="editor-keyboard-shortcut-help__title">
+		<span className="edit-post-keyboard-shortcut-help__title">
 			{ __( 'Keyboard Shortcuts' ) }
 		</span>
 	);
@@ -86,7 +86,7 @@ export function KeyboardShortcutHelpModal( { isModalActive, toggleModal } ) {
 			/>
 			{ isModalActive && (
 				<Modal
-					className="editor-keyboard-shortcut-help"
+					className="edit-post-keyboard-shortcut-help"
 					title={ title }
 					closeLabel={ __( 'Close' ) }
 					onRequestClose={ toggleModal }
diff --git a/edit-post/components/keyboard-shortcut-help-modal/style.scss b/edit-post/components/keyboard-shortcut-help-modal/style.scss
index d656f6011bb3a0..69f77a257f6b81 100644
--- a/edit-post/components/keyboard-shortcut-help-modal/style.scss
+++ b/edit-post/components/keyboard-shortcut-help-modal/style.scss
@@ -1,4 +1,4 @@
-.editor-keyboard-shortcut-help {
+.edit-post-keyboard-shortcut-help {
 	&__title {
 		font-size: 1rem;
 		font-weight: bold;
diff --git a/edit-post/components/keyboard-shortcut-help-modal/test/__snapshots__/index.js.snap b/edit-post/components/keyboard-shortcut-help-modal/test/__snapshots__/index.js.snap
index 32b8afeca7a6e8..207c2570bd2752 100644
--- a/edit-post/components/keyboard-shortcut-help-modal/test/__snapshots__/index.js.snap
+++ b/edit-post/components/keyboard-shortcut-help-modal/test/__snapshots__/index.js.snap
@@ -11,10 +11,16 @@ exports[`KeyboardShortcutHelpModal should match snapshot when the modal is activ
     }
   />
   <WithInstanceId(Modal)
-    className="editor-keyboard-shortcut-help"
+    className="edit-post-keyboard-shortcut-help"
     closeLabel="Close"
     onRequestClose={[Function]}
-    title="Keyboard Shortcuts"
+    title={
+      <span
+        className="edit-post-keyboard-shortcut-help__title"
+      >
+        Keyboard Shortcuts
+      </span>
+    }
   >
     <ShortcutSection
       key="0"
@@ -28,10 +34,6 @@ exports[`KeyboardShortcutHelpModal should match snapshot when the modal is activ
             "description": "Save your changes.",
             "key": "Ctrl+S",
           },
-          Object {
-            "description": "Show or hide the settings sidebar.",
-            "key": "Ctrl+Shift+,",
-          },
           Object {
             "description": "Undo your last changes.",
             "key": "Ctrl+Z",
@@ -40,6 +42,10 @@ exports[`KeyboardShortcutHelpModal should match snapshot when the modal is activ
             "description": "Redo your last undo.",
             "key": "Ctrl+Shift+Z",
           },
+          Object {
+            "description": "Show or hide the settings sidebar.",
+            "key": "Ctrl+Shift+,",
+          },
           Object {
             "description": "Switch between Visual Editor and Code Editor.",
             "key": "Ctrl+Shift+Alt+M",
@@ -101,11 +107,11 @@ exports[`KeyboardShortcutHelpModal should match snapshot when the modal is activ
             "key": "Ctrl+U",
           },
           Object {
-            "description": "Convert the selected text into a link",
+            "description": "Convert the selected text into a link.",
             "key": "Ctrl+K",
           },
           Object {
-            "description": "Remove a link",
+            "description": "Remove a link.",
             "key": "Shift+Alt+S",
           },
           Object {