PR Feedback

- Improve wording in setting descriptions
- Move shared types into common area instead of using type imports
- Fix 'fetchMemory' not returning the correct promise

package.json

@@ -260,7 +260,7 @@
           ],
           "enumDescriptions": [
             "Refresh when the debugger stops (e.g. a breakpoint is hit)",
-            "Refresh automatically when the view is newly focussed.",
+            "Refresh automatically each time the view receives focus.",
             "Refresh automatically after the configured `#memory-inspector.autoRefreshDelay#`",
             "Memory Inspector needs to be manually refreshed by the user"
           ],

src/common/webview-configuration.ts

@@ -0,0 +1,46 @@
+/********************************************************************************
+ * Copyright (C) 2024 EclipseSource and others.
+ *
+ * This program and the accompanying materials are made available under the
+ * terms of the Eclipse Public License v. 2.0 which is available at
+ * http://www.eclipse.org/legal/epl-2.0.
+ *
+ * This Source Code may also be made available under the following Secondary
+ * Licenses when the conditions for such availability set forth in the Eclipse
+ * Public License v. 2.0 are satisfied: GNU General Public License, version 2
+ * with the GNU Classpath Exception which is available at
+ * https://www.gnu.org/software/classpath/license.html.
+ *
+ * SPDX-License-Identifier: EPL-2.0 OR GPL-2.0 WITH Classpath-exception-2.0
+ ********************************************************************************/
+import { WebviewIdMessageParticipant } from 'vscode-messenger-common';
+import { AutoRefresh, Endianness, GroupsPerRowOption } from './manifest';
+import { Radix } from './memory-range';
+
+/** The memory display configuration that can be specified for the memory widget. */
+export interface MemoryDisplayConfiguration {
+    bytesPerMau: number;
+    mausPerGroup: number;
+    groupsPerRow: GroupsPerRowOption;
+    endianness: Endianness;
+    scrollingBehavior: ScrollingBehavior;
+    addressPadding: AddressPadding;
+    addressRadix: Radix;
+    showRadixPrefix: boolean;
+    autoRefresh: AutoRefresh;
+    autoRefreshDelay: number;
+}
+
+export type ScrollingBehavior = 'Paginate' | 'Grow' | 'Auto-Append';
+
+export type AddressPadding = 'Minimal' | number;
+
+export interface ColumnVisibilityStatus {
+    visibleColumns: string[];
+}
+
+/** All settings related to memory view that can be specified for the webview from the extension "main". */
+export interface MemoryViewSettings extends ColumnVisibilityStatus, MemoryDisplayConfiguration {
+    title: string
+    messageParticipant: WebviewIdMessageParticipant;
+}

src/plugin/memory-webview-main.ts

@@ -46,8 +46,8 @@ import {
     WriteMemoryResult,
     writeMemoryType,
 } from '../common/messaging';
+import { MemoryViewSettings, ScrollingBehavior } from '../common/webview-configuration';
 import { getVisibleColumns, WebviewContext } from '../common/webview-context';
-import type { MemoryViewSettings, ScrollingBehavior } from '../webview/utils/view-types';
 import { isVariablesContext } from './external-views';
 import { outputChannelLogger } from './logger';
 import { MemoryProvider } from './memory-provider';

src/webview/memory-webview-view.tsx

@@ -303,7 +303,7 @@ class App extends React.Component<{}, MemoryAppState> {
             // may happen when we initialize empty
             return;
         }
-        this.doFetchMemory(completeOptions);
+        return this.doFetchMemory(completeOptions);
     };
 
     protected async doFetchMemory(memoryOptions: Required<MemoryOptions>): Promise<void> {

src/webview/utils/view-types.ts

@@ -16,11 +16,11 @@
 
 import deepequal from 'fast-deep-equal';
 import type * as React from 'react';
-import { WebviewIdMessageParticipant } from 'vscode-messenger-common';
-import { AutoRefresh, Endianness, GroupsPerRowOption } from '../../common/manifest';
+import { Endianness } from '../../common/manifest';
 import { Memory } from '../../common/memory';
-import { areRangesEqual, BigIntMemoryRange, Radix } from '../../common/memory-range';
+import { areRangesEqual, BigIntMemoryRange } from '../../common/memory-range';
 import { ReadMemoryArguments } from '../../common/messaging';
+import { MemoryDisplayConfiguration } from '../../common/webview-configuration';
 
 export interface SerializedTableRenderOptions extends MemoryDisplayConfiguration {
     columnOptions: Array<{ label: string, doRender: boolean }>;
@@ -73,39 +73,13 @@ export interface FullNodeAttributes extends StylableNodeAttributes {
     content: string;
 }
 
-/** All settings related to memory view that can be specified for the webview from the extension "main". */
-export interface MemoryViewSettings extends ColumnVisibilityStatus, MemoryDisplayConfiguration {
-    title: string
-    messageParticipant: WebviewIdMessageParticipant;
-}
-
-/** The memory display configuration that can be specified for the memory widget. */
-export interface MemoryDisplayConfiguration {
-    bytesPerMau: number;
-    mausPerGroup: number;
-    groupsPerRow: GroupsPerRowOption;
-    endianness: Endianness;
-    scrollingBehavior: ScrollingBehavior;
-    addressPadding: AddressPadding;
-    addressRadix: Radix;
-    showRadixPrefix: boolean;
-    autoRefresh: AutoRefresh;
-    autoRefreshDelay: number;
-}
-export type ScrollingBehavior = 'Paginate' | 'Grow' | 'Auto-Append';
-
-export type AddressPadding = 'Minimal' | number;
 export const AddressPaddingOptions = {
     'Minimal': 'Minimal',
     'Unpadded': 0,
     '32bit': 32,
     '64bit': 64,
 } as const;
 
-export interface ColumnVisibilityStatus {
-    visibleColumns: string[];
-}
-
 export type ReactInteraction<E extends Element = Element> = React.MouseEvent<E> | React.KeyboardEvent<E>;
 
 export function isTrigger(event: ReactInteraction): boolean {