updatees to fix ui issues

Author: ddooley

docs/README_schema_editor.md

@@ -129,10 +129,26 @@ The Schema Editor lets you copy one or more rows from any tab into a different s
    - Right-click **within the highlighted selection** to preserve the multi-row selection while opening the context menu.
 3. Right-click the selection and choose **Copy to schema**.
 4. In the dialog that appears:
-   - Choose the **target schema** from the drop-down list. The schema(s) the selected rows belong to are shown as disabled and cannot be chosen as a target.
-   - Review the **Dependencies** list, if shown. Any structural items that must accompany the selection (see below) are listed here.
+   - Choose the **target schema** from the **Copy to schema** drop-down. The source schema(s) are shown as disabled and cannot be chosen as a target.
+   - Review the three optional sections described below.
 5. Click **Copy**. The rows are appended to the appropriate tabs of the target schema, and a confirmation message reports how many rows were written.
 
+#### Dialog sections
+
+**Section 1 — Selected rows and their parents (always copied)**
+This section is always visible. The selected rows are listed first, grouped under their tab name. Below them, any structural items they depend on that are not yet present in the target schema (e.g. a Table row that a Field refers to, or a base slot that a slot_usage customises) are listed. Everything in this section is always copied and cannot be individually deselected.
+
+**Section 2 — Dependent table records (optional)**
+Child records that belong to the selected rows — for example, Field rows that belong to a selected Table, or Enum value rows that belong to a selected Enum — are listed here. Uncheck the section to skip copying them entirely.
+
+**Section 3 — Copy picklists and picklist choices (optional)**
+Enumerations (picklists) referenced by the `range` attribute of selected or subordinate fields are listed here. Each enumeration has its own checkbox; expanding an enumeration shows a checkbox for each individual permissible value. All items are checked by default.
+
+- Uncheck an **enumeration** to skip that entire picklist and all its values.
+- Uncheck individual **permissible value** rows to copy the enumeration but exclude specific choices.
+
+> **Note:** If you choose not to copy a picklist that one or more fields reference in their `range`, those fields will show a validation error on the Field tab (broken enumeration reference). This is expected — you can resolve it later by either copying the missing picklist separately or updating the field's `range` to point to a picklist that already exists in the target schema.
+
 ### What gets copied automatically (dependencies)
 
 Before appending the selected rows, the editor checks whether any structural items they reference are already present in the target schema. Items that are missing are copied along automatically:

lib/AppContext.js

@@ -148,8 +148,14 @@ export default class AppContext {
 
     //console.log("REFRESHTABDISPLAY", class_name, this.schemaEditor, report_type)
 
-    // Display Focus path only if there is some path > 1 element.
-    $("#record-hierarchy-div").toggle(Object.keys(this.dhs).length > 1);
+    // Show Display menu and breadcrumb only when the current class has inter-table
+    // relations (parent FK slots or child tables).  A standalone class with neither
+    // has no use for "Record(s) by selected key" filtering or a hierarchy path.
+    const classRel = this.relations?.[class_name];
+    const hasRelations = !!classRel
+      && (Object.keys(classRel.parent || {}).length > 0
+          || Object.keys(classRel.child || {}).length > 0);
+    $("#record-hierarchy-div").toggle(hasRelations);
 
     // Record hierarchy cookie crumbs on Display line at top of toolbar are
     // only shown for "Records by 1-many key"
@@ -591,8 +597,10 @@ export default class AppContext {
     this.dhs = this.makeDHsFromRelations(schema, template_name);
 
     // this.currentDataHarmonizer is now set.
-    this.crudGetDependentRows(this.current_data_harmonizer_name);
-    this.crudUpdateRecordPath();
+    // Use crudCalculateDependentKeys (not just crudGetDependentRows) so that
+    // setDHTabStatus() is also called and tabs are correctly enabled/disabled
+    // from the start — without requiring a user cell-click to trigger it.
+    this.crudCalculateDependentKeys(this.current_data_harmonizer_name);
 
     // Ensure dynamic Enums are added so picklist source / range references are
     // detected and pulldown/multiselect menus are crafted; and validation works
@@ -1559,9 +1567,13 @@ export default class AppContext {
     // Presumes dependent_rows has been updated:
     const domId = Object.keys(this.dhs).indexOf(class_name);
     const state = this.dependent_rows.get(class_name).fkey_status;
+    // Disable when state < 2: state 0 = no FK satisfied, state 1 = partial
+    // (some FKs met, some not — e.g. schema_id known but class_id unknown).
+    // Enable when state 2 (no FKs) or 3 (all FKs satisfied).
+    const disabled = state < 2;
     $('#tab-data-harmonizer-grid-' + domId)
-      .toggleClass('disabled',state === 0)
-      .parent().toggleClass('disabled',state === 0);
+      .toggleClass('disabled', disabled)
+      .parent().toggleClass('disabled', disabled);
   }
 
   /** 

lib/DataHarmonizer.js

@@ -218,6 +218,7 @@ class DataHarmonizer {
         copyPaste: true,
         outsideClickDeselects: false, // for maintaining selection between tabs
         manualColumnResize: true,
+        manualRowResize: false, // Note Handsontable has a bug according to Claude: What's happening: When the context menu opens (right-click), the ManualRowResize plugin's _onContextMenu handler tries to clean up its resize-guide DOM element by calling parent.removeChild(element). But a HOT re-render (triggered by, e.g., hidden-rows plugin recalculating, or switching selection across a tab with many rows) has already detached and re-created that DOM node, so the parent no longer holds the original node reference — causing the NotFoundError. 
         //colWidths: [100], //Just fixes first column width
         minRows: 5,
         minSpareRows: 0,
@@ -363,6 +364,16 @@ class DataHarmonizer {
               },
               callback() { self.context.schemaEditor.previewSchema(); }
             },
+            copy_to_schema: {
+              name: 'Copy to schema',
+              hidden() {
+                const schema = self.context.dhs.Schema;
+                return schema?.schema.name !== "DH_LinkML";
+              },
+              callback() {
+                self.context.schemaEditor.openCopyToSchemaModal(self);
+              }
+            },
             // FUTURE Implementation
             // Issue is that this doesn't freeze the column user is on.  It 
             // freezes the first unfrozen column to left.  
@@ -945,6 +956,14 @@ class DataHarmonizer {
           const wtHolder = self.hot.rootElement?.querySelector('.wtHolder');
           const scrollTop = wtHolder?.scrollTop ?? 0;
           self.hot.loadData(reordered);
+          // loadData() resets the hiddenRows plugin, revealing all rows.
+          // In "Records by selected key" mode, re-apply the tab filter so only
+          // rows matching the current key selection remain visible.
+          // In "All records" mode all rows were already visible, so no re-apply
+          // is needed (and doing so would trigger unwanted cross-tab side effects).
+          if (($('input[name="display-main-type"]:checked').val() ?? '') !== 'all') {
+            self.context?.refreshTabDisplay();
+          }
           setTimeout(() => {
             if (wtHolder) wtHolder.scrollTop = scrollTop;
             // Re-select exactly the moved rows without auto-scrolling the
@@ -1226,6 +1245,8 @@ class DataHarmonizer {
     if (isEmpty(parents)) {
       // Insert the new rows below the last existing row
       this.hot.alter(row_where, startRowIndex, numRows);
+      // hot.alter() can reset HOT's hidden-columns plugin state; re-apply concise view.
+      if (this._conciseViewActive) this.context.toolbar?._applyConciseView(this);
       return;
     }
 
@@ -1257,6 +1278,8 @@ class DataHarmonizer {
         });
       }
     });
+    // hot.alter() can reset HOT's hidden-columns plugin state; re-apply concise view.
+    if (this._conciseViewActive) this.context.toolbar?._applyConciseView(this);
   }
 
   /**