Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Suggestions box's first choice is sometimes selected, sometimes NOT #173387

Closed
iuliust opened this issue Feb 3, 2023 · 35 comments
Closed

Suggestions box's first choice is sometimes selected, sometimes NOT #173387

iuliust opened this issue Feb 3, 2023 · 35 comments
Assignees
Labels
feature-request Request for new features or functionality on-release-notes Issue/pull request mentioned in release notes polish Cleanup and polish issue snippets suggest IntelliSense, Auto Complete verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@iuliust
Copy link

iuliust commented Feb 3, 2023

Type: Bug

Since last update, auto-completion is completely broken.

As I type a few characters, the suggestions box appears, but more ofter than not (and contrary to expectation), none of the box's proposals is selected : I have to press the down arrow to select the 1st line!

Consequence:
if I don't pay attention to whether the first choice is pre-selected, I hit ENTER if the first result looks like it's what I want to insert, but since nothing was actually selected, this creates a new line instead... This is probably a bug because this is a terrible UX.

Aside from that, thank you so much for your hard work, VS Code is such a outstanding product!

VS Code version: Code 1.75.0 (e2816fe, 2023-02-01T15:23:45.584Z)
OS version: Windows_NT x64 10.0.22000
Modes:
Sandboxed: Yes

System Info
Item Value
CPUs Intel(R) Core(TM) i7-10750H CPU @ 2.60GHz (12 x 2592)
GPU Status 2d_canvas: enabled
canvas_oop_rasterization: disabled_off
direct_rendering_display_compositor: disabled_off_ok
gpu_compositing: enabled
multiple_raster_threads: enabled_on
opengl: enabled_on
rasterization: enabled
raw_draw: disabled_off_ok
skia_renderer: enabled_on
video_decode: enabled
video_encode: enabled
vulkan: disabled_off
webgl: enabled
webgl2: enabled
webgpu: disabled_off
Load (avg) undefined
Memory (System) 31.64GB (5.05GB free)
Process Argv --crash-reporter-id 7dc5599a-5121-4310-803e-9045cdb3905e
Screen Reader no
VM 0%
Extensions (20)
Extension Author (truncated) Version
codesnap adp 1.3.4
ng-template Ang 15.1.0
markdown-mermaid bie 1.17.4
npm-intellisense chr 1.4.4
vscode-markdownlint Dav 0.49.0
vscode-eslint dba 2.2.6
gitlens eam 13.2.0
EditorConfig Edi 0.16.4
vscode-test-explorer hbe 2.21.1
angular2-switcher inf 0.4.0
karma-test-explorer luc 0.8.1
dotenv mik 1.0.1
vscode-docker ms- 1.23.3
vscode-language-pack-fr MS- 1.75.2023020109
remote-containers ms- 0.275.0
remote-wsl ms- 0.75.1
test-adapter-converter ms- 0.1.6
vsliveshare ms- 1.0.5828
material-icon-theme PKi 4.23.1
vscode-npm-scripts tra 0.2.1
A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627:30244334
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vscaat:30438848
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
azure-dev_surveyone:30548225
pyindex848cf:30577861
nodejswelcome1cf:30587006
2e4cg342:30602488
pyind779:30657576
f6dab269:30613381
pythonsymbol12:30657548
vscsb:30628654

@KunalTanwar
Copy link

KunalTanwar commented Feb 4, 2023

Same Issue!!

Before 1.75.0 it autofocuses on the suggestions we just need to use Arrow keys for the desired one but now it is not auto focusing and if we press any Arrow key the cursor just shifts to the above or below line hiding the dropdown suggestions.

@ArturoDent
Copy link

Check this new setting:

Editor > Suggest: Selection Mode

Is that set to always?

@KunalTanwar
Copy link

Yes it is set to always

@dalbitresb12
Copy link

Related #173343.

I've also checked that option and it is also always, but the first choice is still not always selected.

@jrieken
Copy link
Member

jrieken commented Feb 6, 2023

Does this occur when invoking a completion when just having accepted one, esp. a snippet completion? Do you have the editor.suggest.snippetsPreventQuickSuggestions setting configured?

@KunalTanwar
Copy link

Does this occur when invoking a completion when just having accepted one, esp. a snippet completion? Do you have the editor.suggest.snippetsPreventQuickSuggestions setting configured?

Yes it is set to true but still same problem sometimes working sometimes not!

@dalbitresb12
Copy link

I think the real problem here is that the suggestion box is showing up when using snippets with editor.suggest.snippetsPreventQuickSuggestions set to true (which is the default). It should only show up when it is set to false.

I've done some testing using Windows Sandbox with versions 1.74.3 and 1.75.0. With "editor.suggest.snippetsPreventQuickSuggestions": false, the first option is selected in both versions. With "editor.suggest.snippetsPreventQuickSuggestions": true (or unset), in 1.74.3 the box doesn't show while in 1.75.0 it shows up with the first option not selected.

You can see both behaviors in the following videos:

1.74.3
Code-1.74.3.mp4
1.75.0
Code-1.75.0.mp4

@jrieken
Copy link
Member

jrieken commented Feb 7, 2023

in 1.74.3 the box doesn't show while in 1.75.0 it shows up with the first option not selected.

That change was done on purpose but it wasn't clearly communicated. In the past we would truly suppress Intellisense when in snippet mode. Over the last years this has caused much confusion and heated debates because people believe IntelliSense is broken. The underlying conflict is that both (snippets and suggest) use Tab to either advance to next placeholder or accept a completion. They are mutually exclusive and one is given precedence making folks believe things are broken.

Since IntelliSense has the new "no select" mode I figured it is a good compromise to use that instead of suppressing suggest. Wrt muscle memory nothing should have changed, pressing Tab still goes to the next snippet placeholder but wrt the UI are now seeing the suggest widget (as ambient offering)

@jrieken jrieken added the under-discussion Issue is under discussion for relevance, priority, approach label Feb 7, 2023
@jrieken
Copy link
Member

jrieken commented Feb 7, 2023

/cc @waynebloss

@ghost
Copy link

ghost commented Feb 7, 2023

I am not sure what the correct workaround should be, but I think that things started working again when I set snippetsPreventQuickSuggestions to false.

I searched around for a "no select" Intellisense setting but I couldn't find one. (Is this a reference to editor.suggest.selectionMode and setting it to never?)

Basically, I guess I just want it to work the way that it always had worked: Whether I'm using a snippet or not, when I see the autocomplete/intellisense popup list the first item should always be highlighted and when I press Tab, it completes the selected item and moves to the next snippet placeholder if in snippet mode...

Thanks!

@fabioz
Copy link
Contributor

fabioz commented Feb 8, 2023

Is there any way to revert to the previous mode since it was done "on purpose"?

Personally, I find the new way extremely annoying (and given that it always was this way, I think it should default to the way it always was and an option could turn on the new option which doesn't select completions even though they're being shown when on snippet mode).

@fabioz
Copy link
Contributor

fabioz commented Feb 8, 2023

As a note, it seems that setting "editor.suggest.snippetsPreventQuickSuggestions": false does improve it for me (at least when a completion is shown it can be applied).

But now I'm confused... when "editor.suggest.snippetsPreventQuickSuggestions": true, the completions are actually being shown but without marking a completion item as the selected item (so, it doesn't prevent quick suggestions, it makes it so that it shows it but requires the user to use Ctrl+Space again to focus it, which is not good IMHO -- definitely not good as a default, and that seems to be the bug here... in 1.74.3 the completions wouldn't be shown in that case, so, there was no confusion, the new way of handling "editor.suggest.snippetsPreventQuickSuggestions": true to show completions but not select it is very confusing).

Maybe it'd be better if it'd be controlled by something as:

"editor.suggest.onSnippetMode": "showCompletionsButDontSelect" (current behavior -- seems strange to me... maybe it could be there but not default)
"editor.suggest.onSnippetMode": "dontShowCompletions" (old behavior -- maybe not ideal, as users think completions don't work but is reasonable for those preferring placeholders over completions?)
"editor.suggest.onSnippetMode": "showCompletionsAndApplyOnTabAndEnter" (old behavior when "editor.suggest.snippetsPreventQuickSuggestions": false was active -- it can backfire because pressing tab wouldn't go to the next snippet and would apply the completion, but maybe that'd be a better default as there wouldn't be a confusion that code-completion isn't working).
"editor.suggest.onSnippetMode": "showCompletionsAndApplyOnEnter" (I guess that'd be the ideal setting for me -- as I use enter to apply completions and not tab, but for users using tab to apply completions I don't really see a way out anyways as the conflict is there by design).

I understand there's a conflict that needs to be solved where tab can be used both to apply a completion and go to the next placeholder and that maybe a setting can help there, but showing a completion and not letting it be applied makes it confusing IMHO and shouldn't be the default.

@jrieken
Copy link
Member

jrieken commented Feb 10, 2023

@fabioz spot on. The setting should be extended to support the new mode. I missed that because my reasoning was the default didn't change wrt the keypress sequences. When in snippet mode, pressing Tab wouldn't insert a suggestion (because suppressed/invisible 🤓) and it still doesn't. However, the confusing part that has changed is that the widget shows but isn't active. The default will likely be what it is today. The snippet suppressing has confused v e r y many folks over the years and we have received countless of issues and questions why IntelliSense is "broken" when editing snippets. This change is meant to bring clarity for that but needs more tweaking

@jrieken jrieken added feature-request Request for new features or functionality polish Cleanup and polish issue and removed under-discussion Issue is under discussion for relevance, priority, approach labels Feb 10, 2023
@fabioz
Copy link
Contributor

fabioz commented Feb 10, 2023

@jrieken note that there is no conflict when a user uses tab to go to the next placeholder and enter to apply a completion (so, if the completions are there I'd definitely expect enter to apply it and tab to go to the next placeholder...).

@daguitosama
Copy link

editor.suggest.snippetsPreventQuickSuggestions : false

This setting fixed the issue for me, big thanks.
At this point I think is pretty hard to not press Enter when the suggestion widget shows up. That Enter almost press itself.

@jrieken
Copy link
Member

jrieken commented May 9, 2023

When you make stable release?

This will ship early June

@michaeldaw

This comment was marked as outdated.

@michaeldaw
Copy link

This is what is happening for me in 1.79-insiders:

clip01.mp4

@michaeldaw
Copy link

I just read this comment in #183411
image

Frankly, I find that remark inflammatory.

I've been using various IDE's for nearly 20 years. You can always use TAB to confirm a completion suggestion. Always. Whatever the original intent of this change was, it is an unmitigated failure. Restore the original functionality.

@rami3l
Copy link

rami3l commented May 27, 2023

As someone from the "Team Tab" I second @michaeldaw's comment.

As a matter of fact, editor.suggest.snippetsPreventQuickSuggestions being true by default has already caused confusion for people like me and I have been using VSCode with this value set to false since I'm aware of this option.

Removing this option might be good, but I'm aware of it exactly because I want to use Tab completions in snippets.

Personally I don't mind jumping to another completion point "manually".
Better yet, maybe the hot key for jumping to the next completion point should be changed in the case of hot key conflicts, as suggested in https://stackoverflow.com/a/76335930.

@roblourens roblourens added the verified Verification succeeded label May 30, 2023
@jrieken
Copy link
Member

jrieken commented May 31, 2023

We will revisit this

@jrieken jrieken reopened this May 31, 2023
@jrieken jrieken removed the verified Verification succeeded label May 31, 2023
@jrieken
Copy link
Member

jrieken commented May 31, 2023

#183954 is up for more refinements/changes:

  1. reverts the TAB changes (undoes what was said here)
  2. revert the focus behaviours (the cause of this issue)
  3. flips the default for the snippetsPreventQuickSuggestions setting

The latter isn't guaranteed to happen and we might revisit this. It's not that we are making changes in thin air, usually things start with users reporting an issue and it's sometimes hard to judge how many folks share the pain or are happy. So, please bear with us

@jrieken jrieken closed this as completed Jun 1, 2023
@jrieken
Copy link
Member

jrieken commented Jun 1, 2023

To verify

  • ensure the snippetsPreventQuickSuggestions setting is back and its default is false
  • ensure typing inside a placeholder of a snippet shows suggestions, ensure TAB accepts it
  • ensure ESC dismisses suggestion and that snippet mode stays on (until hitting ESC again)
  • turn snippetsPreventQuickSuggestions to true and ensure typing in a snippet placeholder doesn't trigger quick suggestions

@hediet hediet added verified Verification succeeded verification-found Issue verification failed and removed verified Verification succeeded labels Jun 1, 2023
@hediet
Copy link
Member

hediet commented Jun 1, 2023

turn snippetsPreventQuickSuggestions to true and ensure typing in a snippet placeholder doesn't trigger quick suggestions

Does not work for me:

Code_-_Insiders_XGunqwfNfA

@hediet hediet added verified Verification succeeded and removed verification-found Issue verification failed labels Jun 1, 2023
jrieken added a commit that referenced this issue Jun 1, 2023
jrieken added a commit that referenced this issue Jun 1, 2023
@jrieken jrieken added the on-release-notes Issue/pull request mentioned in release notes label Jun 2, 2023
kburtram added a commit to microsoft/azuredatastudio that referenced this issue Jun 27, 2023
* log when an editor action doesn't run because of enablement

* notebooks create/dispose editors. this means controllers must be created eagerly (😢) and that notebooks need a custom way of plugging comparision keys for session. works unless creating another session for the same cell of a duplicated editor

* Set offSide to sql lang configuration to true (#183461)

* Fixes #181764 (#183550)

* fix typo

* Always scroll down and focus the input (#183557)

* Fixes #180386 (#183561)

* cli: ensure ordering of rpc server messages (#183558)

* cli: ensure ordering of rpc server messages

Sending lots of messages to a stream would block them around the async
tokio mutex, which is "fair" so doesn't preserve ordering. Instead, use
the write_loop approach I introduced to the server_multiplexer for the
same reason some time ago.

* fix clippy

* update for May endgame

* testing: allow invalidateTestResults to take an array (#183569)

* Document `ShareProvider` API proposal (#183568)

* Document `ShareProvider` API proposal

* Remove mention of VS Code from JSDoc

* Add support for rendering svg and md in welcome message (#183580)

* Remove toggle setting more eagerly (#183584)

* rm message abt macOS

* Change text (#183589)

* Change text

* Accidentally changed the wrong file

* cli: improve output for code tunnel status (#183571)

* testing: allow invalidateTestResults to take an array

* cli: improve output for code tunnel status

Fixes #183570

* [json/css/html] update services (#183595)

* Add experimental setting to enable this dialog

* Fix exporting chat model to JSON before it is initialized (#183597)

* minimum scrolling to reveal the next cell on shift+enter (#183600)

do minimum scrolling to reveal the next cell on Execute cell and select next

* Fixing Jupyter notebook issue 13263 (#183527)

fix for the issue, still need to understand why there is strange focusing

* Tweak proposed API JSDoc (#183590)

* Tweak proposed API JSDoc

* workbench -> workspace

* fix ? operator

* Use active editor and show progress when sharing (#183603)

Use active editor and show progress

* use scroll setting variable correctly

* Schedule welcome widget to show once between typing. (#183606)

* Schedule dialog to show once between typing

* Don't re-render if already displayed once

* Add F10 keybinding for debugger step, even on Web. (#183510)

Fixes #181792.
Previously, for Web the keyboard shortcut was Alt-F10, because it was
believed that F10 could not be bound on browsers. This turned out to be
incorrect, so we make the shortcut consistent (F10) with desktop VSCode
which is also what many other debuggers use.
We keep Alt-F10 on web as a secondary keybinding to keep the experience
some web users may have gotten used to by now.

* Also pass process.env

* Restore missing chat clear commands (#183651)

* chore: update electron@22.5.4 (#183716)

* Show remote indicator in web when remoteAuthority is set (#183728)

* feat: .vuerc as json file (#153017)

Co-authored-by: Martin Aeschlimann <martinae@microsoft.com>

* Delete --compatibility=1.63 code from the server (#183738)

* Copy vscode.dev link to tunnel generates an invalid link when an untitled workspace is open (#183739)

* Recent devcontainer display string corrupted on Get Started page (#183740)

* Improve "next codeblock" navigation (#183744)

* Improve "next codeblock" navigation
Operate on the current focused response, or the last one, and scroll to the selected item

* Normalize command title

* Git - run git status if similarityThreshold changes (#183762)

* fix aria-label issue in kb editor

fixes A11y_GradeB_VSCode_Keyboard shortcut reads words together - Blind: Arrow key navigation to row Find the binding keys and  "when" cell data are read together resulting in a word " CTRL + FeditorFocus  instead of CTRL + F editorFocus" #182490

* Status - fix compact padding (#183768)

* Remove angle brackets from VB brackets (#183782)

Fixes #183359

* Update language config schema with more details about brackets. (#183779)

* fix comment (#183812)

* Support for `Notebook` CodeAction Kind (#183457)

* nb kind support -- wip

* allow notebook codeactions around single cell edit check

* move notebook code action type out of editor

---------

Co-authored-by: rebornix <penn.lv@gmail.com>

* cli: fix connection default being applied (#183827)

* cli: bump to openssl 1.1.1u (#183828)

* Implement "delete" action for chat history (#183609)

* Use desired file name when generating new md pasted file paths (#183861)

Fixes #183851

* Default to filename for markdown new file if empty (#183864)

Fixes #183848

* Fix small typo (#183865)

Fixes #183819

* Noop when moving a symbol into the file it is already in (#183866)

Fixes #183793

* Adjust codeAction validation to account for notebook kind (#183859)

* Make JS/TS `go to configuration` commands work on non-`file:` file systems (#183688)

Make `go to project` commands work on non-`file:` file systems

Fixes #183685

* Can't do regex search after opening notebook (#183884)

Fixes #183858

* Default to current dir for `move to file` select (#183875)

Fixes #183870

`showOpenDialog` seems to ignore `defaultUri` if the file doesn't exist

* Use `<...>` style markdown links when needed (#183876)

Fixes #183849

* Remove check for context keys

* Update xterm package

* Enable updating a chat model without triggering incremental typing (#183894)

* Enable chat "move" commands on empty sessions (#183895)

* Enable chat "move" commands on empty sessions
and also imported sessions

* Fix command name

* Fix some chat keybindings on windows (#183896)

* "Revert File" on inactive editors are ignored (fix #177557) (#183903)

* Empty reason while switching profile (fix #183775) (#183904)

* fix https://github.com/microsoft/vscode-internalbacklog/issues/4278 (#183910)

* fix microsoft/vscode#183770 (#183914)

* code --status displays a lot of errors before actual status output (fix #183787) (#183915)

* joh/icy manatee (#183917)

* Use idle value for widget of interactive editor controller

microsoft/vscode#183820

* also make preview editors idle values

microsoft/vscode#183820

* Fix #183777 (#183929)

* Fix #182309 (#183925)

* Tree checkbox item -> items (#183931)

Fixes #183826

* Fixes #183909 (#183940)

* Fix #183837 (#183943)

fix #183837

* Git - fix #183941 (#183944)

* Update xterm.css

Fixes #181242

* chore: add @ulugbekna and @aiday-mar to my-endgame notebook (#183946)

* Revert "When snippet mode is active, make `Tab` not accept suggestion but advance placeholder"

This reverts commit 50a80cd.

* revert not focusing completion list when quick suggest happens during snippet

* change `snippetsPreventQuickSuggestions` default to false

* Fix #181446 (#183956)

* fix https://github.com/microsoft/vscode-internalbacklog/issues/4298 (#183957)

* fix: remove extraneous incorrect context keys (#183959)

These were actually getting added in getTestItemContextOverlay, and the test ID was using the extended ID which extensions do not know about.

Fixes #183612

* Fixes microsoft/monaco-editor#3920 (#183960)

* fix https://github.com/microsoft/vscode-internalbacklog/issues/4324 (#183961)

* fix #183030

* fix #180826 (#183962)

* make message more generic for interactive editor help

* .

* fix #183968

* Keep codeblock toolbar visible when focused

* Fix when clause on "Run in terminal" command

* add important info to help menu

* fix #183970

* Set `isRefactoring` for all TS refactoring edits (#183982)

* consolidate

* Disable move to file in TS versions < 5.2 (#183992)

There are still a few key bugs with refactoring. We will  ship this as a preview for TS 5.2+ instead of for 5.1

* Polish query accepting (#183995)

We shouldn't send the same request to Copilot if the query hasn't changed. So if the query is the same, we short circut.

Fixes https://github.com/microsoft/vscode-internalbacklog/issues/4286

Also, when we open in chat, we should use the last accepted query, not what's in the input box.

Fixes https://github.com/microsoft/vscode-internalbacklog/issues/4280

* Allow widget to have focus (#184000)

So that selecting non-code text works.

Fixes https://github.com/microsoft/vscode-internalbacklog/issues/4294

* Fix microsoft/vscode-internalbacklog#4257. Mitigate zindex for zone widgets. (#184001)

* Change welcome dialog contribution to Eventually

* Misc fixes

* Workspace folder picker entry descriptions are suboptimal for some filesystems (fix #183418) (#184018)

* cli - ignore std error unless verbose (#183787) (#184031)

* joh/inquisitive meerkat (#184034)

* only stash sessions that are none empty

https://github.com/microsoft/vscode-internalbacklog/issues/4281

* only unstash a session once - unless new exchanges are made,

https://github.com/microsoft/vscode-internalbacklog/issues/4281

* account for all exchange types

* Improve declared components (#184039)

* make sure to read setting (#184040)

d'oh, related to microsoft/vscode#173387 (comment)

* [html] update service (#184049)

[html] update service. FIxes #181176

* reset context keys on reset/hide (#184042)

fixes https://github.com/microsoft/vscode-internalbacklog/issues/4330

* use `Lazy`, not `IdleValue` for the IE widget held by the eager controller (#184048)

microsoft/vscode#183820

* fix https://github.com/microsoft/vscode-internalbacklog/issues/4333 (#184067)

* use undo-loop instead of undo-edit when discarding chat session (#184063)

* use undo-loop instead of undo-edit when discarding chat session

fixes https://github.com/microsoft/vscode-internalbacklog/issues/4118

* fix tests, wait for correct state

* Add logging to node download (#184070)

Add logging to node download. For #182951

* re-enable default zone widget revealing when showing (#184072)

fixes https://github.com/microsoft/vscode-internalbacklog/issues/4332, also fixes https://github.com/microsoft/vscode-internalbacklog/issues/3784

* fix #178202

* Allow APIs in stable (#184062)

* Fix microsoft/vscode-internalbacklog#4206. Override List view whitespace css for monaco editor (#184087)

* Fix JSDoc grammatical error (#184090)

* Pick up TS 5.1.3 (#184091)

Fixes #182931

* Misc fixes

* update distro (#184097)

* chore: update electron@22.5.5 (#184116)

* Extension host veto is registered multiple times on restart (fix #183778) (#184127)

Extension host veto is registered multiple times on restart (#183778)

* Do not auto start the local web worker extension host (#184137)

* Allow embedders to intercept trustedTypes.createPolicy calls (#184136)

Allow embedders to intercept trustedTypes.createPolicy calls (#184100)

* fix: reading from console output for --status on windows and linux (#184138)

fix: reading from console output for --status on windows and linux (#184118)

* Misc fixes

* code --status displays a lot of errors before actual status output (fix #183787) (#184200)

fix 183787

* (cherry-pick to 1.79 from main) Handle galleryExtension failure in featuredExtensionService (#184205)

Handle galleryExtension failure in featuredExtensionService (#184198)

Handle galleryExtension failure

* Fix #184183. Multiple output height updates are skipped. (#184188)

* Post merge init fixes

* Misc build issues

* disable toggle inline diff of `alt` down

https://github.com/microsoft/vscode-internalbacklog/issues/4342

* Take into account already activated extensions when computing running locations (#184303)

Take into account already activated extensions when computing running locations (fixes #184180)

* Avoid `extensionService.getExtension` and use `ActivationKind.Immediate` to allow that URI handling works while resolving (#184310)

Avoid `extensionService.getExtension` and use `ActivationKind.Immediate` to allow that URI handling works while resolving (fixes #182217)

* WIP

* rm fish auto injection

* More breaks

* Fix Port Attributes constructor (#184412)

* WIP

* WIP

* Allow extensions to get at the exports of other extensions during resolving (#184487)

Allow extensions to get at the exports of other extensions during resolving (fixes #184472)

* do not auto finish session when inline chat widgets have focus

re https://github.com/microsoft/vscode-internalbacklog/issues/4354

* fix compile errors caused by new base method

* WIP

* WIP

* WIP

* WIP

* Build errors

* unc - fix path traversal bypass

* Bump version

* cherry-pick prod changes from main

* Disable sandbox

* Build break from merge

* bump version

* Merge pull request #184739 from max06/max06/issue184659

Restore ShellIntegration for fish (#184659)

* Git - only add --find-renames if the value is not the default one (#185053)

Git - only add --find-renames if the value is not the default one (#184992)

* Cherry-pick: Revert changes to render featured extensions when available (#184747)

Revert changes to render featured extensions when available.  (#184573)

* Lower timeouts for experimentation and gallery service

* Revert changes to render extensions when available

* Add audio cues

* fix: disable app sandbox when --no-sandbox is present (#184913)

* fix: disable app sandbox when --no-sandbox is present (#184897)

* fix: loading minimist in packaged builds

* Runtime errors

* UNC allow list checks cannot be disabled in extension host (fix #184989) (#185085)

* UNC allow list checks cannot be disabled in extension host (#184989)

* Update src/vs/base/node/unc.js

Co-authored-by: Robo <hop2deep@gmail.com>

---------

Co-authored-by: Robo <hop2deep@gmail.com>

* Add notebook extension

* Fix mangling issues

* Fix mangling issues

* npm install

* npm install

* Issues blocking bundle

* Fix build folder compile errors

* Fix windows bundle build

* Linting fixes

* Fix sqllint issues

* Update yarn.lock files

* Fix unit tests

* Fix a couple breaks from test fixes

* Bump distro

* redo the checkbox style

* Update linux build container dockerfile

* Bump build image tag

* Bump native watch dog package

* Bump node-pty

* Bump distro

* Fix documnetation error

* Update distro

* redo the button styles

* Update datasource TS

* Add missing yarn.lock files

* Windows setup fix

* Turn off extension unit tests while investigating

* color box style

* Remove appx

* Turn off test log upload

* update dropdownlist style

* fix universal app build error (#23488)

* Skip flaky bufferContext vscode test

---------

Co-authored-by: Johannes <johannes.rieken@gmail.com>
Co-authored-by: Henning Dieterichs <hdieterichs@microsoft.com>
Co-authored-by: Julien Richard <jairbubbles@hotmail.com>
Co-authored-by: Charles Gagnon <chgagnon@microsoft.com>
Co-authored-by: Megan Rogge <merogge@microsoft.com>
Co-authored-by: meganrogge <megan.rogge@microsoft.com>
Co-authored-by: Rob Lourens <roblourens@gmail.com>
Co-authored-by: Connor Peet <connor@peet.io>
Co-authored-by: Joyce Er <joyce.er@microsoft.com>
Co-authored-by: Bhavya U <bhavyau@microsoft.com>
Co-authored-by: Raymond Zhao <7199958+rzhao271@users.noreply.github.com>
Co-authored-by: Martin Aeschlimann <martinae@microsoft.com>
Co-authored-by: Aaron Munger <aamunger@microsoft.com>
Co-authored-by: Aiday Marlen Kyzy <amarlenkyzy@microsoft.com>
Co-authored-by: rebornix <penn.lv@gmail.com>
Co-authored-by: Ole <oler@google.com>
Co-authored-by: Jean Pierre <jeanp413@hotmail.com>
Co-authored-by: Robo <hop2deep@gmail.com>
Co-authored-by: Yash Singh <saiansh2525@gmail.com>
Co-authored-by: Ladislau Szomoru <3372902+lszomoru@users.noreply.github.com>
Co-authored-by: Ulugbek Abdullaev <ulugbekna@gmail.com>
Co-authored-by: Alex Ross <alros@microsoft.com>
Co-authored-by: Michael Lively <milively@microsoft.com>
Co-authored-by: Matt Bierner <matb@microsoft.com>
Co-authored-by: Andrea Mah <31675041+andreamah@users.noreply.github.com>
Co-authored-by: Benjamin Pasero <benjamin.pasero@microsoft.com>
Co-authored-by: Sandeep Somavarapu <sasomava@microsoft.com>
Co-authored-by: Daniel Imms <2193314+Tyriar@users.noreply.github.com>
Co-authored-by: Tyler James Leonhardt <me@tylerleonhardt.com>
Co-authored-by: Alexandru Dima <alexdima@microsoft.com>
Co-authored-by: Joao Moreno <Joao.Moreno@microsoft.com>
Co-authored-by: Alan Ren <alanren@microsoft.com>
@github-actions github-actions bot locked and limited conversation to collaborators Jul 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality on-release-notes Issue/pull request mentioned in release notes polish Cleanup and polish issue snippets suggest IntelliSense, Auto Complete verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

No branches or pull requests