Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Misleading variable name #4432

Open
Rick-Kirkham opened this issue Feb 21, 2024 · 0 comments
Open

Misleading variable name #4432

Rick-Kirkham opened this issue Feb 21, 2024 · 0 comments
Assignees
Labels
Area: PowerPoint Feedback on PowerPoint content Needs: attention 👋 Waiting on Microsoft to provide feedback Type: doc bug Problem with the documentation (e.g., doc is out of date, unclear, confusing, or broken)

Comments

@Rick-Kirkham
Copy link
Contributor

Rick-Kirkham commented Feb 21, 2024

The code sample in the Delete shapes section has these two lines:

const sheet = context.presentation.slides.getItemAt(0);
const shapes = sheet.shapes;

The variable sheet is probably copied from an Excel example. It should be slide.


Document Details

Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs: triage 🔍 New issue, needs PM on rotation to triage ASAP label Feb 21, 2024
@Rick-Kirkham Rick-Kirkham added Area: PowerPoint Feedback on PowerPoint content Type: doc bug Problem with the documentation (e.g., doc is out of date, unclear, confusing, or broken) Needs: attention 👋 Waiting on Microsoft to provide feedback and removed Needs: triage 🔍 New issue, needs PM on rotation to triage ASAP labels Feb 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: PowerPoint Feedback on PowerPoint content Needs: attention 👋 Waiting on Microsoft to provide feedback Type: doc bug Problem with the documentation (e.g., doc is out of date, unclear, confusing, or broken)
Projects
None yet
Development

No branches or pull requests

2 participants