fix: autotagging attributes (#753)

contentful · Jun 4, 2024 · c1a527e · c1a527e
1 parent 4633be5
commit c1a527e
Show file tree

Hide file tree

Showing 8 changed files with 352 additions and 83 deletions.
diff --git a/packages/live-preview-sdk/src/__tests__/saveEvent.spec.ts b/packages/live-preview-sdk/src/__tests__/saveEvent.spec.ts
@@ -24,7 +24,10 @@ describe('SaveEvent', () => {
     const callback = vi.fn();
     const entry = { sys: { id: '1' } } as Entry;
 
-    const saveEvent = new SaveEvent({ locale });
+    const saveEvent = new SaveEvent({
+      locale,
+      options: { locale: 'en-US', targetOrigin: ['http://localhost:1234'] },
+    });
     saveEvent.subscribe(callback);
     saveEvent.receiveMessage({
       data: {},
@@ -40,7 +43,10 @@ describe('SaveEvent', () => {
     const callback = vi.fn();
     const entry = { sys: { id: '2' } } as Entry;
 
-    const saveEvent = new SaveEvent({ locale });
+    const saveEvent = new SaveEvent({
+      locale,
+      options: { locale: 'en-US', targetOrigin: ['http://localhost:1234'] },
+    });
     saveEvent.subscribe(callback);
     saveEvent.receiveMessage({
       method: LivePreviewPostMessageMethods.ENTRY_SAVED,

diff --git a/packages/live-preview-sdk/src/index.ts b/packages/live-preview-sdk/src/index.ts
@@ -153,7 +153,15 @@ export class ContentfulLivePreview {
 
       if (this.liveUpdatesEnabled) {
         this.liveUpdates = new LiveUpdates({ locale, targetOrigin: this.targetOrigin });
-        this.saveEvent = new SaveEvent({ locale });
+        this.saveEvent = new SaveEvent({
+          locale,
+          options: {
+            locale: this.locale,
+            space: this.space,
+            environment: this.environment,
+            targetOrigin: this.targetOrigin,
+          },
+        });
       }
 
       // bind event listeners for interactivity
@@ -196,7 +204,13 @@ export class ContentfulLivePreview {
       // tell the editor that there's a SDK
       const { taggedElements, manuallyTaggedCount, automaticallyTaggedCount } = this
         .inspectorModeEnabled
-        ? getAllTaggedElements()
+        ? getAllTaggedElements({
+            options: {
+              locale: this.locale,
+              space: this.space,
+              environment: this.environment,
+            },
+          })
         : {
             taggedElements: [],
             manuallyTaggedCount: 0,
@@ -367,7 +381,13 @@ export class ContentfulLivePreview {
    * Returns a list of tagged entries on the page
    */
   static getEntryList(): string[] {
-    return getAllTaggedEntries();
+    return getAllTaggedEntries({
+      options: {
+        locale: this.locale,
+        space: this.space,
+        environment: this.environment,
+      },
+    });
   }
 }
 

diff --git a/packages/live-preview-sdk/src/inspectorMode/__tests__/getAllTaggedElements.bench.ts b/packages/live-preview-sdk/src/inspectorMode/__tests__/getAllTaggedElements.bench.ts
@@ -89,7 +89,11 @@ describe('getAllTaggedElements', () => {
         </div>
       `);
 
-    const elements = getAllTaggedElements(dom, true);
+    const elements = getAllTaggedElements({
+      root: dom,
+      ignoreManual: true,
+      options: { locale: 'en-US' },
+    });
 
     expect(elements).toHaveLength(1 + 10 + 30 + 3);
   });