Remove hard-coded zebra in running extensions Fixes #40432

[usernamehw]

#40432

The same way it's already done in keybindings editor.

Also, there is another issue in "running extensions": .focused selector not overwriting .odd in dark and high contrast themes - it was broken before me.

[usernamehw]

On the second thought... why it's even using typescript+css for striping and not :nth-child(odd)?

[usernamehw]

And why it's using .even in the keybindings and .odd in running extensions?

[isidorn]

@usernamehw all are fair suggestions feel free to add them to this PR and let me know when you are done so I review. Thanks

[usernamehw]

There was another issue: no hover on list items.

And another: click on list element usually applies "list.inactiveFocusBackground", but not in the "running extensions" list.

---

CSS nth-child(odd) is different from .odd.

.odd - index starting with 0
nth-child(odd) - index starting with 1

So it named now as even but behave exactly as it was before (in running extensions).

Changed from odd(as in CSS) to even in keybindings though... Because the first line merges with header.

Before:

After:

@isidorn

[usernamehw]

Vendor prefixes from other browsers: are those legacy from monaco-editor or something else? You can search with this regex: ^\s+(-moz|-ms|-khtml|-o). Many of the -webkit ones are obsolete too.

[isidorn]

@usernamehw thanks for your PR I have reveiwed the runtime extensions changes and this looks good to me 👍
As for the changes in the keybindings editor, assigning to @sandy081 to review. If he gives the green light we can merge this.
Just for future we prefer to have seperate PRs for seperate feature areas.
As for the monaco-editor we are still shipping the monaco-editor as a sepearte product and it supports other browsers, so that is why we keep other browsers in some areas.

[sandy081]

LGTM

[isidorn]

@usernamehw thanks a lot 🍻