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

Insert item #589

Merged
merged 69 commits into from
May 4, 2020
Merged

Insert item #589

merged 69 commits into from
May 4, 2020

Conversation

nilshedstrom
Copy link
Contributor

This is an implementation of the functionallity described in #508

It allows you to insert

  • Parameters
  • Variables
  • Functions
  • Resources
  • Outputs

The functionallity can be called by

  • Command pallette: Insert Item...
  • Editor context menu: Insert Item...
  • ARM template outline context menu: Insert Item...
  • ARM template outline context menu: Insert Parameter... (only for parameter nodes)
  • ARM template outline context menu: Insert Variable... (only for variable nodes)
  • ARM template outline context menu: Insert Function... (only for function nodes)
  • ARM template outline context menu: Insert Resource... (only for resource nodes)
  • ARM template outline context menu: Insert Output... (only for output nodes)

Insert Function will always insert the new function in the first namespace (if it exists) otherwise it will create a new namespace.

Insert Resource will use the existing snippets to create a new resource but the user will be able to select which resource from a list that can be filtered.

Nils Hedström and others added 30 commits December 13, 2019 14:48
changed so that all items in TreeView has the same context as the root Item.
@StephenWeatherford
Copy link
Contributor

FYI, you don't need to feel rushed, but but any features for the next release to be merged by early next week so we can schedule testing. Thanks. If not, that's fine, it'll go in the next release.

@StephenWeatherford
Copy link
Contributor

Mostly this looks ready after the changes I suggested. We might play with the UI a bit afterwards, but nothing major I don't think.

Copy link
Contributor

@StephenWeatherford StephenWeatherford left a comment

Choose a reason for hiding this comment

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

Looks great, again much thanks!

A couple of minor bugs/fit-and-finish issues found after playing with it, entered those separately because I want to get this merged now.

#669 (should probably be fixed now)
#670 (lower priority items)

I'll fix #669 tomorrow or Wednesday if you can't or don't want to get to it, no worries.

@StephenWeatherford StephenWeatherford merged commit 99c5a10 into microsoft:master May 4, 2020
@StephenWeatherford StephenWeatherford mentioned this pull request May 5, 2020
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