Skip to content

Add PDF viewer documentation#10793

Open
cindylay wants to merge 2 commits intoSharePoint:mainfrom
cindylay:cindylay/pdfchanges
Open

Add PDF viewer documentation#10793
cindylay wants to merge 2 commits intoSharePoint:mainfrom
cindylay:cindylay/pdfchanges

Conversation

@cindylay
Copy link
Copy Markdown
Contributor

Category

  • Content fix
  • New article

What's in this Pull Request?

This PR adds a new PDF Preview section to the Using File Previews article, documenting the query parameters available for enhancing the SharePoint Embedded static PDF previewer.

Changes

  • Added ## PDF Preview section to using-file-preview.md introducing the embed query parameter mechanism, including:
    • An explanation of what webUrl is and how to retrieve it from the driveItem Graph endpoint
    • A syntax example showing how to combine multiple parameters in a single embed JSON object
    • A note that additional parameters will be documented as the PDF previewer grows
    • Print (mpp) — documents the mpp boolean parameter that enables the print icon and Ctrl+P functionality
    • Sticky Notes (mpsn) — documents the mpsn boolean parameter that shows sticky note content in a PDF
  • Updated ms.date to reflect the current revision date

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 53cc837:

✅ Validation status: passed

File Status Preview URL Details
docs/embedded/development/tutorials/using-file-preview.md ✅Succeeded View

For more details, please refer to the build report.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds documentation for enhancing SharePoint Embedded’s PDF preview experience by appending embed query parameters to a DriveItem webUrl in the existing “Using File Previews” tutorial.

Changes:

  • Updated ms.date to the current revision date.
  • Added a new “PDF Preview” section describing an embed query parameter mechanism.
  • Documented two PDF preview parameters: Print (mpp) and Sticky Notes (mpsn).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/embedded/development/tutorials/using-file-preview.md Outdated
Comment on lines +111 to +113
```text
<webUrl>&embed={"<param1>":<value>,"<param2>":<value>}
```
Copy link

Copilot AI Apr 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The syntax example uses <webUrl>&embed=..., which only works if webUrl already has a ? query string. Consider documenting the correct delimiter (?embed= when there are no existing query params, otherwise &embed=) to avoid readers building invalid URLs.

Copilot uses AI. Check for mistakes.
Comment on lines +124 to +126
```text
<webUrl>&embed={"mpp":true}
```
Copy link

Copilot AI Apr 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The embed value contains JSON with quotes/braces that should be URL-encoded when used in a query string (e.g., via encodeURIComponent), otherwise the example URL is not valid in many clients/browsers. Update the examples to show an encoded embed value (or explicitly state that the JSON must be URL-encoded).

Copilot uses AI. Check for mistakes.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit d8966a9:

✅ Validation status: passed

File Status Preview URL Details
docs/embedded/development/tutorials/using-file-preview.md ✅Succeeded View

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@cindylay cindylay requested a review from dluces April 22, 2026 16:31
@andrewconnell andrewconnell self-assigned this Apr 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants