-
Notifications
You must be signed in to change notification settings - Fork 70
Add transition to the slide #338
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
Conversation
|
Learn Build status updates of commit d640c67:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| docs/presentation/how-to-add-transitions-between-slides-in-a-presentation.md | Details | ||
| samples/presentation/add_transition/cs/add_transition_cs.csproj | ✅Succeeded | ||
| samples/presentation/add_transition/cs/Program.cs | ✅Succeeded | ||
| samples/presentation/add_transition/vb/add_transition_vb.vbproj | ✅Succeeded | ||
| samples/presentation/add_transition/vb/Program.vb | ✅Succeeded | ||
| samples/samples.sln | ✅Succeeded |
docs/presentation/how-to-add-transitions-between-slides-in-a-presentation.md
- Line 67, Column 64: [Warning: xref-not-found - See documentation]
Cross reference not found: 'DocumentFormat.OpenXml.Presentation.Transition*'. - Line 94, Column 50: [Warning: xref-not-found - See documentation]
Cross reference not found: 'DocumentFormat.OpenXml.Presentation.Linq.P14*'.
For more details, please refer to the build report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:
- Try searching the learn.microsoft.com contributor guides
- Post your question in the Learn support channel
|
Learn Build status updates of commit 5b7e978:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| docs/presentation/how-to-add-transitions-between-slides-in-a-presentation.md | Details | ||
| samples/presentation/add_transition/cs/add_transition_cs.csproj | ✅Succeeded | ||
| samples/presentation/add_transition/cs/Program.cs | ✅Succeeded | ||
| samples/presentation/add_transition/vb/add_transition_vb.vbproj | ✅Succeeded | ||
| samples/presentation/add_transition/vb/Program.vb | ✅Succeeded | ||
| samples/samples.sln | ✅Succeeded |
docs/presentation/how-to-add-transitions-between-slides-in-a-presentation.md
- Line 94, Column 50: [Warning: xref-not-found - See documentation]
Cross reference not found: 'DocumentFormat.OpenXml.Presentation.Linq.P14'.
For more details, please refer to the build report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:
- Try searching the learn.microsoft.com contributor guides
- Post your question in the Learn support channel
|
Learn Build status updates of commit d7e1138: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
|
Learn Build status updates of commit 9523230: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
|
Learn Build status updates of commit 46072f4: ❌ Validation status: errorsPlease follow instructions here which may help to resolve issue.
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. For any questions, please:
|
|
Learn Build status updates of commit 1f1ec9c:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| docs/presentation/overview.md | Details | ||
| docs/presentation/how-to-add-transitions-between-slides-in-a-presentation.md | ✅Succeeded | ||
| docs/toc.yml | ✅Succeeded | ||
| samples/presentation/add_transition/cs/add_transition_cs.csproj | ✅Succeeded | ||
| samples/presentation/add_transition/cs/Program.cs | ✅Succeeded | ||
| samples/presentation/add_transition/vb/add_transition_vb.vbproj | ✅Succeeded | ||
| samples/presentation/add_transition/vb/Program.vb | ✅Succeeded | ||
| samples/samples.sln | ✅Succeeded |
docs/presentation/overview.md
- Line 54, Column 3: [Warning: file-not-found - See documentation]
Invalid file link: 'presentation/how-to-add-transitions-between-slides-in-a-presentation.md'.
For more details, please refer to the build report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:
- Try searching the learn.microsoft.com contributor guides
- Post your question in the Learn support channel
|
Learn Build status updates of commit 9ae8728: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
| { | ||
| // Get all AlternateContent elements | ||
| List<AlternateContent> alternateContents = [.. slidePart.Slide.Descendants<AlternateContent>()]; | ||
| foreach (var alternateContent in alternateContents) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's use types here instead of var to be consistent
Co-authored-by: Michael Bowen <10384982+mikeebowen@users.noreply.github.com>
|
Learn Build status updates of commit d757a68: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
|
Learn Build status updates of commit fa8ce02: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
|
Learn Build status updates of commit 46f6b5c: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
|
@mkaszewiak @mikeebowen Hi, I fixed the merge conflict and approved. Please add one other reviewer to merge this. When you're ready to merge, please @ me in comments. Thanks, Linda |
|
@twsouthwick or @tomjebo as Mikee is OOF could you please approve it. Thanks |
No description provided.