Danny Tuppeny

Test gutter icons/locations do not update correctly when extension replaces children

4

This came up in the Dart extension, but I can reproduce in [test-provider-example](https://github.com/microsoft/vscode-extension-samples/tree/main/test-provider-sample). - Clone the vscode-extension-samples project and open test-provider-sample - Comment out the line `data.updateFromContents(ctrl, e.getText(), file);` in...

When the host saves a file, it triggers code actions on the guest which never complete

6

This was mentioned in https://github.com/MicrosoftDocs/live-share/issues/3504, but that's been tagged as a feature request and asks about having settings to enable/disable. However, I think this is a more serious bug. If...

Hide Code Runner button for .dart files if Dart extension is also installed

1

Hi! I work on the [Dart extension](https://marketplace.visualstudio.com/items?itemName=Dart-Code.dart-code) for VS Code. I've recently added buttons to make running/debugging Dart files more discoverable (based on recent [VS Code recommendations](https://code.visualstudio.com/updates/v1_47#_single-file-debugging)) though uses with...

See discussion at https://github.com/Dart-Code/Dart-Code/issues/3975#issuecomment-1234507375. Empty `Container()`s may trigger lints, and `Placeholder` seems like an ideal replacement (`const Placeholder()`). I'm going to change this in VS Code + LSP (https://github.com/Dart-Code/Dart-Code/issues/4136) and...

Renaming a class by its constructor doesn't work

3

Pressing `F2` to rename on a constructor like this doesn't do anything when you press `` (no error, no rename).

```dart class A extends B {} class B {} ``` When browsing the hierarchy from A, we see a node for `B` but it doesn't include `` (or ``).

`extends` here isn't coloured right. This might also be fixed by changes in progress in #4067.

I would expect all `ignores` to be further down, and the "add import" to further up. Not sure if it's VS Code's new menu or a server issue.

Add the ability to run tests for a folder to explorer context menu

2

"Run all tests in folder" / "Debug all tests in folder" To match it, we should probably also rename "Start Debugging"/"Run without debugging" to include "tests" when on a test...

The new DAPs don't handle the "inspect" event that was used to navigate the editor to code when inspecting widgets. Perhaps it's worth considering a more specific event for navigating...