Update jcad exporting mechanism (#355)

* Make convert endpoint configurable

* Fix edge selection

* Update Playwright Snapshots

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

Author: trungleduc

packages/base/src/3dview/mainview.tsx

@@ -515,7 +515,6 @@ export class MainView extends React.Component<IProps, IStates> {
     const selectedMeshesNames = new Set(
       this._selectedMeshes.map(sel => sel.name)
     );
-
     if (selection) {
       const selectionName = selection.mesh.name;
       if (e.ctrlKey) {
@@ -540,7 +539,6 @@ export class MainView extends React.Component<IProps, IStates> {
         newSelection[name] = this._meshGroup?.getObjectByName(name)
           ?.userData as ISelection;
       }
-
       this._updateSelected(newSelection);
       this._model.syncSelected(newSelection, this._mainViewModel.id);
     }

packages/base/src/commands.ts

@@ -593,8 +593,13 @@ const EXPORT_FORM = {
   },
   syncData: (context: DocumentRegistry.IContext<IJupyterCadModel>) => {
     return (props: IDict) => {
+      const endpoint = context.model?.sharedModel?.toJcadEndpoint;
+      if (!endpoint) {
+        showErrorMessage('Error', 'Missing endpoint.');
+        return;
+      }
       const { Name } = props;
-      requestAPI<{ done: boolean }>('jupytercad/export', {
+      requestAPI<{ done: boolean }>(endpoint, {
         method: 'POST',
         body: JSON.stringify({
           path: context.path,
@@ -895,9 +900,9 @@ export function addCommands(
   commands.addCommand(CommandIDs.exportJcad, {
     label: trans.__('Export to .jcad'),
     isEnabled: () => {
-      return tracker.currentWidget
-        ? tracker.currentWidget.context.model.sharedModel.exportable
-        : false;
+      return Boolean(
+        tracker.currentWidget?.context?.model?.sharedModel?.toJcadEndpoint
+      );
     },
     iconClass: 'fa fa-file-export',
     execute: async () => {

packages/base/src/panelview/objectproperties.tsx

@@ -207,13 +207,8 @@ class ObjectPropertiesReact extends React.Component<IProps, IStates> {
     if (newState) {
       const selection = newState.selected.value;
       const selectedObjectNames = Object.keys(selection || {});
-
       // Only show object properties if ONE object is selected and it's a shape
-      if (
-        selection === undefined ||
-        selectedObjectNames.length !== 1 ||
-        selection[selectedObjectNames[0]].type !== 'shape'
-      ) {
+      if (selection === undefined || selectedObjectNames.length !== 1) {
         this.setState(old => ({
           ...old,
           schema: undefined,
@@ -223,7 +218,14 @@ class ObjectPropertiesReact extends React.Component<IProps, IStates> {
         return;
       }
 
-      const selectedObject = selectedObjectNames[0];
+      let selectedObject = selectedObjectNames[0];
+      if (
+        selection[selectedObject] &&
+        selection[selectedObject].type !== 'shape'
+      ) {
+        selectedObject = selection[selectedObject].parent as string;
+      }
+
       if (selectedObject !== this.state.selectedObject) {
         const objectData = this.props.cpModel.jcadModel?.getAllObject();
         if (objectData) {

packages/schema/src/doc.ts

@@ -187,7 +187,6 @@ export class JupyterCadDoc
   }
 
   editable = true;
-  exportable = false;
 
   private _getObjectAsYMapByName(name: string): Y.Map<any> | undefined {
     for (const obj of this._objects) {

packages/schema/src/interfaces.ts

@@ -85,8 +85,9 @@ export interface IJupyterCadDoc extends YDocument<IJupyterCadDocChange> {
   options: JSONObject;
   metadata: JSONObject;
   outputs: JSONObject;
+
   readonly editable: boolean;
-  readonly exportable: boolean;
+  readonly toJcadEndpoint?: string;
 
   objectExists(name: string): boolean;
   getObjectByName(name: string): IJCadObject | undefined;

python/jupytercad_core/src/stepplugin/model.ts

@@ -50,7 +50,7 @@ export class JupyterCadStepDoc extends JupyterCadDoc {
   }
 
   editable = false;
-  exportable = true;
+  toJcadEndpoint = 'jupytercad/export';
 
   private _sourceObserver = (events: Y.YEvent<any>[]): void => {
     const changes: Array<{