Gavin Barron

Results 176 comments of Gavin Barron

You can then add that captured variable to your pipeline using this `echo "##vso[task.setvariable variable=packageVersion]$version" [Documentation](https://docs.microsoft.com/en-us/azure/devops/pipelines/process/variables?view=azure-devops&tabs=yaml%2Cbatch&viewFallbackFrom=vsts#set-a-job-scoped-variable-from-a-script)

Browser support for `::part()` is pretty good now, https://caniuse.com/?search=%3A%3Apart , and probably the best way we have outside of more CSS variables.

@musale this looks beautiful, but I can't use my keyboard to select anything in the list now :(

Can we expand a group when we get hits within it due to a search? Having the groups collapsed is a bit of a regression from the previous behavior. ![image](https://user-images.githubusercontent.com/7122716/186466624-c4bf4ec5-53f2-4a78-b372-2051f17352ce.png)

Nice work @musale, It's getting better with the keyboard, now I can tab from the input field to the filtered tree view. Can we add a key down handler that...

It looks like web component analyzer will support what we need to make this fly in the future based on: https://github.com/runem/web-component-analyzer/issues/169 and https://github.com/runem/web-component-analyzer/pull/157 I've raised https://github.com/runem/web-component-analyzer/issues/248 to see if this...

@Mnickii I've assigned this to you as you're the best person to assess how well the new person card component supports templating of the sub-components.

For the people picker directly, no it's not needed AFAIK. However, to render the selected people we user the `mgt-person` which in turn uses the `mgt-person-card` which on the organization...

Based on my test it appears to work for `lit^2.3.1` We're just in the process of upgrading to lit v2 and use this library. Our usage of the analyzer does...

Looks like the overflow case for the left-hand pane occurs when the screen is < 1500px wide. I think our best course of action here is to get some telemetry...