While working on the tree control for our extension, I noticed that the sample implementation was not optimal. For instance, `vscode.TreeDataProvider.onDidChangeTreeData` should only be called once for the top-most node that has changed, as all children will be refreshed as well.
This change updates the code of the sample to track state changes during propagation and only raise the event for the top-most parent node that has its state changing.
There also was an issue with root node `1` not rendering, which impacted the new algorithm, as it was not refreshing the tree, when invisible root was changing. In order to address this, I updated `getChildren` to actually return the `1` node, when root was requested. This in turn changed the presentation (`1` is now rendered in the tree) and addressed the problem with tree not refreshing when `1` is the node that raises the event.
Lastly, there is one more optimization that I didn't make due to the bug that is in the works: `getTreeItem` can actually return the `element` directly, without the copy, as it already implements `TreeComponentItem`. The problem currently is that `TreeNode` also defines an `id`, which causes an error during refresh. Once that is addressed - this part can be optimized as well.
* Made azure arc as resourcesubtype
Added new azure arc resource type
Added support for different eula statement for different subtypes
Consolidated getSelectedOption
* Fixed some PR based comments
* Fixed more pr comments
* Fixed the error in unit test by deep copying extension resourceTypes (to keep the original one intact)
* Fixed property name 'agreement' to 'agreements'
* Cloning subresourceTypes
* fix for start and stop xevent sessions
* vscode.open for help URL
* setup info messages for localization
@kburtram - I could use an assist on updating the v# and publishing the vsix, but there's no rush this can wait until after the holidays
* save not yet tested work
* Merge from master.
* Screeens Shared for Feeedback
* Code complete
* remove unneeded changes
* remove unnecessary comma
* remov wss
* remove dead code
* PR feedback
* checkpoint fixes
* PR & minor fixes
* minor fix for feature of resourceType options being optional.
* reverting experimental change
* separating out changes for future featurework.
* revert unneeded change
* review feedback fixes
* review feedback
* rename InputFieldComponent to InputComponent
* working version of custom summary page
* add option to align items in a flex- container.
* changes to support labelColor
* save work , still pending issue with labelCSSStyles
* Summary page and setting variabless in notebook.
* minor fixes.
* pr feedbck
* fix formatting issues
* pr feedback
* pr feedback
* pr feedback
* fixing docs
* summary page value setting fix
* rename children of RowInfo to items
* rename a method
* rename summary_text to evaluated_text
* rename properties of fieldInfo
* revert inadvertent change
* rename linked_texttext to hyperlinked_text and removing linking facility from readonly_text
* pr feedback
* fix setting tools variables in env and notebook
* removing saving of originalValues for EvaluatedText
* await on launchNotebookWithEdits
* await on launchNotebookWithContent
* merge RadioOptions & Options into 1
* merge ReadOnlyText, links & evaluatedText
* Samples for new generic wizard features
* fix comment
* fix assertions
* return type and comment for getClusterContext
* fix inadvertent change
* increase minimum required azdata version
* remove unneeded environment variable settings
* not leaking passwords in notebooks
* update packages
* remove fs
* fix more deps
* test adding governance to ci
* ignore samples for cacheing
* fix save cache too
* bump more packages
* attempt to fix slickgrid
* add more deps
* use slickgrid version
* bump https-proxy
* bump more packages.
* fix service-downloader
* remove typings
* fix compile
* update more packages
* add vscode back to the extensions that need it for testing
* add fail on critical
* regrab proxy agent
* lock more deps
* replace final high warning
* revert service downloader changes
* remove the deps on jquery ui and event drag