💬 fix command name, 'Show/Hide Al' to 'Show/Hide All Tables' to avoid confusion with 'Show All Fields' command

Author: tnyo43

.changeset/breezy-yaks-arrive.md

@@ -2,6 +2,6 @@
 "@liam-hq/erd-core": patch
 ---
 
-- ✨ Add "Show All" and "Hide All" table visibility shortcuts and options in CommandPalette
-  - Command Palette: new "Show All" and "Hide All" commands
-  - Subscribe shortcuts: ⇧A for "Show All" and ⇧H for "Hide All"
+- ✨ Add "Show All Tables" and "Hide All Tables" table visibility shortcuts and options in CommandPalette
+  - Command Palette: new "Show All Tables" and "Hide All Tables" commands
+  - Subscribe shortcuts: ⇧A for "Show All Tables" and ⇧H for "Hide All Tables"

frontend/packages/erd-core/src/features/erd/components/ERDRenderer/CommandPalette/CommandPaletteOptions/CommandOptions.test.tsx

@@ -50,7 +50,7 @@ it('renders options with descriptions', async () => {
 })
 
 describe('show/hide all tables options', () => {
-  it('shows "Show All" option and hide "Hide all" option when all tables are hidden', () => {
+  it('shows "Show All Tables " option and hide "Hide All Tables" option when all tables are hidden', () => {
     mockDefaultNodes.mockReturnValueOnce([
       {
         id: '1',
@@ -71,14 +71,14 @@ describe('show/hide all tables options', () => {
     render(<CommandPaletteCommandOptions />, { wrapper })
 
     expect(
-      screen.getByRole('option', { name: 'Show All ⇧ A' }),
+      screen.getByRole('option', { name: 'Show All Tables ⇧ A' }),
     ).toBeInTheDocument()
     expect(
-      screen.queryByRole('option', { name: 'Hide All ⇧ H' }),
+      screen.queryByRole('option', { name: 'Hide All Tables ⇧ H' }),
     ).not.toBeInTheDocument()
   })
 
-  it('shows "Hide All" option and hide "Show all" option when all tables are visible', () => {
+  it('shows "Hide All Tables" option and hide "Show All Tables" option when all tables are visible', () => {
     mockDefaultNodes.mockReturnValueOnce([
       {
         id: '1',
@@ -99,14 +99,14 @@ describe('show/hide all tables options', () => {
     render(<CommandPaletteCommandOptions />, { wrapper })
 
     expect(
-      screen.queryByRole('option', { name: 'Show All ⇧ A' }),
+      screen.queryByRole('option', { name: 'Show All Tables ⇧ A' }),
     ).not.toBeInTheDocument()
     expect(
-      screen.getByRole('option', { name: 'Hide All ⇧ H' }),
+      screen.getByRole('option', { name: 'Hide All Tables ⇧ H' }),
     ).toBeInTheDocument()
   })
 
-  it('shows both "Show All" and "Hide All" options when some tables are visible and the others are hidden', () => {
+  it('shows both "Show All Tables" and "Hide All Tables" options when some tables are visible and the others are hidden', () => {
     mockDefaultNodes.mockReturnValueOnce([
       {
         id: '1',
@@ -127,10 +127,10 @@ describe('show/hide all tables options', () => {
     render(<CommandPaletteCommandOptions />, { wrapper })
 
     expect(
-      screen.getByRole('option', { name: 'Show All ⇧ A' }),
+      screen.getByRole('option', { name: 'Show All Tables ⇧ A' }),
     ).toBeInTheDocument()
     expect(
-      screen.getByRole('option', { name: 'Hide All ⇧ H' }),
+      screen.getByRole('option', { name: 'Hide All Tables ⇧ H' }),
     ).toBeInTheDocument()
   })
 })

frontend/packages/erd-core/src/features/erd/components/ERDRenderer/CommandPalette/CommandPaletteOptions/CommandOptions.tsx

@@ -117,7 +117,7 @@ export const CommandPaletteCommandOptions: FC = () => {
           }}
         >
           <Eye className={styles.itemIcon} />
-          <span className={styles.itemText}>Show All</span>
+          <span className={styles.itemText}>Show All Tables</span>
           <span className={styles.keyIcon}>⇧</span>
           <span className={styles.keyIcon}>A</span>
         </Command.Item>
@@ -132,7 +132,7 @@ export const CommandPaletteCommandOptions: FC = () => {
           }}
         >
           <EyeOff className={styles.itemIcon} />
-          <span className={styles.itemText}>Hide All</span>
+          <span className={styles.itemText}>Hide All Tables</span>
           <span className={styles.keyIcon}>⇧</span>
           <span className={styles.keyIcon}>H</span>
         </Command.Item>

frontend/packages/erd-core/src/features/erd/components/ERDRenderer/CommandPalette/CommandPalettePreview/CommandPreview.tsx

@@ -5,8 +5,7 @@ type Props = {
   commandName: string
 }
 
-// TODO: set gif or image for "Show All" and "Hide All" commands
-
+// TODO: set gif or image for "Show All Table" and "Hide All Table" commands
 const COMMAND_VIDEO_SOURCE: Record<string, string> = {
   'copy link':
     'https://assets.liambx.com/erd-core/2025-09-01/videos/copy-link.mp4',