Skip to content

Conversation

@DefaultRyan
Copy link
Member

There are a couple big changes to the VSIX in this PR:

Lay foundation for VSIX localization

(Copied this technique from microsoft/WindowsAppSDK#1341)

There is an undocumented method for localizing string resources in VS project and item templates. Firstly, the extension containing the templates must provide a VSPackage. Next, a VSPackage.resx (Visual Studio first looks for the VSPackage resource stream) is added to the extension project to hold the string resources; the IDs of these resources must be integers (specifically, within the range of a unsigned int). Finally, the Name and Description elements in each .vstemplate's <TemplateData> section so that:

  1. They do not provide a text value
  2. The Package attribute is the GUID assigned to the extension's VSPackage
  3. The ID attribute is the ID of the string resource as defined in the .resx

The convention that we will use for choosing resource IDs is as follows:

  • Numbering starts at 1000. This gives us what should be a sufficient number of digits to work around the VS's .resx designer use of a lexical sort.

Remove Dev15 props/targets build cruft

Since the release of Dev16 (Visual Studio 2019), the props and targets that provide build support have shipped in the NuGet package, and are no longer needed in the VSIX. This change removes these unneeded files.

@DefaultRyan
Copy link
Member Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@DefaultRyan
Copy link
Member Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link

@evelynwu-msft evelynwu-msft left a comment

Choose a reason for hiding this comment

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

:shipit:

@DefaultRyan DefaultRyan merged commit 4d5c5ae into master Dec 2, 2021
@DefaultRyan DefaultRyan deleted the user/defaultryan/localize-vsix branch December 2, 2021 20:04
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