refactor: Use localizable messages. (#458)

* refactor: Use localizable messages.

* chore: Fix post-rebase.

Author: gonfunko

src/actions/clipboard.ts

@@ -9,6 +9,7 @@ import {
   Gesture,
   ShortcutRegistry,
   Events,
+  Msg,
   utils as blocklyUtils,
   clipboard,
   ICopyData,
@@ -104,7 +105,8 @@ export class Clipboard {
    */
   private registerCutContextMenuAction() {
     const cutAction: ContextMenuRegistry.RegistryItem = {
-      displayText: (scope) => `Cut (${getShortActionShortcut('cut')})`,
+      displayText: (scope) =>
+        Msg['CUT_SHORTCUT'].replace('%1', getShortActionShortcut('cut')),
       preconditionFn: (scope) => {
         const ws = scope.block?.workspace;
         if (!ws) return 'hidden';
@@ -199,7 +201,8 @@ export class Clipboard {
    */
   private registerCopyContextMenuAction() {
     const copyAction: ContextMenuRegistry.RegistryItem = {
-      displayText: (scope) => `Copy (${getShortActionShortcut('copy')})`,
+      displayText: (scope) =>
+        Msg['COPY_SHORTCUT'].replace('%1', getShortActionShortcut('copy')),
       preconditionFn: (scope) => {
         const ws = scope.block?.workspace;
         if (!ws) return 'hidden';
@@ -304,7 +307,8 @@ export class Clipboard {
    */
   private registerPasteContextMenuAction() {
     const pasteAction: ContextMenuRegistry.RegistryItem = {
-      displayText: (scope) => `Paste (${getShortActionShortcut('paste')})`,
+      displayText: (scope) =>
+        Msg['PASTE_SHORTCUT'].replace('%1', getShortActionShortcut('paste')),
       preconditionFn: (scope: ContextMenuRegistry.Scope) => {
         let block;
         if (scope.focusedNode instanceof Blockly.Block) {

src/actions/delete.ts

@@ -7,10 +7,12 @@
 import {
   ContextMenuRegistry,
   Gesture,
+  Msg,
   ShortcutRegistry,
   utils as BlocklyUtils,
   LineCursor,
 } from 'blockly';
+import {getShortActionShortcut} from '../shortcut_formatting';
 import * as Constants from '../constants';
 import type {WorkspaceSvg} from 'blockly';
 import {Navigation} from '../navigation';
@@ -102,16 +104,17 @@ export class DeleteAction {
 
     const deleteItem: ContextMenuRegistry.RegistryItem = {
       displayText: (scope) => {
+        const shortcut = getShortActionShortcut(this.deleteShortcutName);
         if (!this.oldContextMenuItem) {
-          return 'Delete block (Del)';
+          return Msg['DELETE_BLOCK'].replace('%1', shortcut);
         }
 
         type DisplayTextFn = (p1: ContextMenuRegistry.Scope) => string;
         // Use the original item's text, which is dynamic based on the number
         // of blocks that will be deleted.
         const oldDisplayText = this.oldContextMenuItem
           .displayText as DisplayTextFn;
-        return oldDisplayText(scope) + ' (Del)';
+        return oldDisplayText(scope) + ` (${shortcut})`;
       },
       preconditionFn: (scope, menuOpenEvent: Event) => {
         const ws = scope.block?.workspace;

src/actions/edit.ts

@@ -4,8 +4,10 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import {ContextMenuRegistry, LineCursor} from 'blockly';
+import {ContextMenuRegistry, LineCursor, Msg} from 'blockly';
 import {Navigation} from 'src/navigation';
+import {getShortActionShortcut} from '../shortcut_formatting';
+import * as Constants from '../constants';
 
 /**
  * Action to edit a block.  This just moves the cursor to the first
@@ -50,7 +52,10 @@ export class EditAction {
    */
   private registerContextMenuAction() {
     const editAboveItem: ContextMenuRegistry.RegistryItem = {
-      displayText: 'Edit Block contents (→︎)',
+      displayText: Msg['EDIT_BLOCK_CONTENTS'].replace(
+        '%1',
+        getShortActionShortcut(Constants.SHORTCUT_NAMES.RIGHT),
+      ),
       preconditionFn: (scope: ContextMenuRegistry.Scope, menuOpenEvent) => {
         if (menuOpenEvent instanceof PointerEvent) return 'hidden';
         const workspace = scope.block?.workspace;

src/actions/enter.ts

@@ -6,6 +6,7 @@
 
 import {
   Events,
+  Msg,
   ShortcutRegistry,
   utils as BlocklyUtils,
   getFocusManager,

src/actions/insert.ts

@@ -6,9 +6,11 @@
 
 import {
   ContextMenuRegistry,
+  Msg,
   ShortcutRegistry,
   utils as BlocklyUtils,
 } from 'blockly';
+import {getShortActionShortcut} from '../shortcut_formatting';
 import * as Constants from '../constants';
 import type {WorkspaceSvg} from 'blockly';
 import {Navigation} from '../navigation';
@@ -69,7 +71,10 @@ export class InsertAction {
   private registerContextMenuAction() {
     const insertAboveItem: ContextMenuRegistry.RegistryItem = {
       displayText: () => {
-        return 'Insert Block (I)';
+        return Msg['INSERT_BLOCK'].replace(
+          '%1',
+          getShortActionShortcut(this.insertShortcutName),
+        );
       },
       preconditionFn: (
         scope: ContextMenuRegistry.Scope,