Single quoted string usage is already enforced everywhere except our tests. Having this inconsistent style can confuse contributors and code generation
Starting with converting over tests in the `extensions` dir
For #271167
This makes it so our built-in extensions can mostly be built using `tsc` on the command line. Previously the extensions were picking up a lot of typing info from the root `node_modules` that meant they weren't truly independent
For #269213
This adds a new eslint rule for `as any` and `<any>({... })`. We'd like to remove almost all of these, however right now the first goal is to prevent them in new code. That's why with this first PR I simply add `eslint-disable` comments for all breaks
Trying to get this change in soon after branching off for release to hopefully minimize disruption during debt week work
* Fix#209158. Add Copy Output and Open Output in Text Editor to Scrollable text output context menu.
* Context menu should be triggered on container
* fix naming
* Replace removeChild with remove
This replaces most uses of `parent.removeChild(child)` with
`child.remove()`.
The two are almost equivalent. The only difference is that
`parent.removeChild(child)` throws if the given node is not a child of
the parent, whereas `child.remove()` never throws. There is no noticable
performance difference. The only reason to use `removeChild` is to
support Internet Explorer, but that’s no longer supported by Monaco
editor.
* Fix broken CSP hash
The script content changed, so the sha256 hash changed too.
* Update src/vs/editor/browser/config/charWidthReader.ts
Co-authored-by: Logan Ramos <lramos15@gmail.com>
---------
Co-authored-by: Logan Ramos <lramos15@gmail.com>
* collapse stack trace
* interaction buttons
* fixup
* place feature behind option
* styling, more cleanup
* test returned link, fix test bug
* test header link, skip header link for old IPython
* padding
* remove inline margin
