Merge pull request #166 from OfficeDev/PPT-specific-API

Use a PPT-specific API for the task pane button

Author: Rick-Kirkham

package-lock.json

@@ -19,13 +19,11 @@ export async function run() {
    */
   await PowerPoint.run(async (context) => {
     const shapes = context.presentation.slides.getItemAt(0).shapes;
-    const shapeOptions = {
-      left: 100,
-      top: 300,
-      height: 300,
-      width: 450,
-    };
-    const textbox = shapes.addTextBox("Hello World!", shapeOptions);
+    const textbox = shapes.addTextBox("Hello World!");
+    textbox.left = 100;
+    textbox.top = 100;
+    textbox.height = 300;
+    textbox.width = 450;
     textbox.name = "GreetingTextbox";
     await context.sync();
   });

src/taskpane/powerpoint.js

@@ -23,7 +23,7 @@ async function getText(): Promise<string> {
     await context.sync();
 
     const textboxShapes = shapes.items.filter(shape => shape.name === "GreetingTextbox");
-      
+
     if (textboxShapes.length > 0) {
       const textFrame = textboxShapes[0].textFrame.load("textRange");
       await context.sync();
@@ -34,7 +34,6 @@ async function getText(): Promise<string> {
     }
   });
 }
-
 export async function runTest(): Promise<void> {
   // Execute taskpane code
   await run();

test/end-to-end/src/powerpoint-test-taskpane.ts

@@ -46,7 +46,7 @@ const OfficeMockData = {
 
 describe("PowerPoint", function () {
   it("Run", async function () {
-    
+
     const pptMock: OfficeMockObject = new OfficeMockObject(PowerPointMockData); // Mocking the host specific namespace
     global.PowerPoint = pptMock as any;
     global.Office = new OfficeMockObject(OfficeMockData) as any; // Mocking the common office-js namespace
@@ -57,4 +57,4 @@ describe("PowerPoint", function () {
     // Check that a text box was added with the correct text
     assert.strictEqual(slide.shapes.items[0].textFrame.textRange.text, "Hello World!");
   });
-});
+});