Focus next item in list after removing breakpoint by ChrisPapp · Pull Request #96287 · microsoft/vscode

@ChrisPapp

@ChrisPapp

isidorn

if (this.isBodyVisible()) {
this.updateSize();
if (this.list) {
const lastFocusIndex = this.list.getFocus()[0];

Choose a reason for hiding this comment

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

lastFocusIndex can be undefined, since getFocus can return an empty array.
Also do not check against equality with -1.

@isidorn

@ChrisPapp thanks for this PR. I left some comments, once you address them we can merge this in.
Assigning to May since we are now in the endgame week and we do not accept any new features this week, only testing.

@ChrisPapp

@isidorn

Works nicely. Thanks for this PR, merging in ☀️