Skip to content

Office2021 #1030

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

Merged
merged 3 commits into from
Sep 20, 2021
Merged

Office2021 #1030

merged 3 commits into from
Sep 20, 2021

Conversation

tomjebo
Copy link
Collaborator

@tomjebo tomjebo commented Sep 20, 2021

Office 2021 updated types:
Label Information part support
PowerPoint Comments support
Excel Named Sheet Views support
Excel Rich Data enhancements
Word Comments Extensibilty part types
Document Tasks support in Word
Drawing Sketchy Shapes support

@@ -245,6 +245,9 @@ private static string NormalizeNamespace(in OpenXmlNamespace ns)
{ "http://schemas.microsoft.com/office/spreadsheetml/2014/11/main", "x16", FileFormatVersions.Office2016 },
{ "http://schemas.microsoft.com/office/spreadsheetml/2015/02/main", "x16r2", FileFormatVersions.Office2016 },
{ "http://schemas.microsoft.com/office/word/2015/wordml/symex", "w16se", FileFormatVersions.Office2016 },
// Named 2013/main/command with "oac" ns prefix to avoid conflict with officedocument/2006/characteristics
{ "http://schemas.microsoft.com/office/drawing/2013/main/command", "oac", FileFormatVersions.Office2016 },
Copy link
Member

Choose a reason for hiding this comment

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

Where'd these come from?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is a new ns prefix and namespace added retro actively to 2016, 2019 and in 2021 to include moniker types that were not included a few years ago.

@@ -454,7 +454,7 @@ public OfficeArtExtensionList(string outerXml) : base(outerXml)
internal override void ConfigureMetadata(ElementMetadata.Builder builder)
{
base.ConfigureMetadata(builder);
builder.SetSchema(112, "extLst");
builder.SetSchema(114, "extLst");
Copy link
Member

Choose a reason for hiding this comment

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

Have you done the back and forth for this? We should really switch it out to use the proper schemas

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

these were offset by retroactive introduction of the moniker types for drawing. I don't think they'll impact the public API surface or at least with any breaking change.

twsouthwick
twsouthwick previously approved these changes Sep 20, 2021
@tomjebo tomjebo merged commit 0b98918 into dotnet:main Sep 20, 2021
@tomjebo tomjebo deleted the office2021 branch October 26, 2021 01:06
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