-
Notifications
You must be signed in to change notification settings - Fork 30k
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
"enablePreview" setting is not discoverable #61582
Comments
Thank you very much.!!!!! |
PR welcome |
@bpasero I would like to pick this up! |
@karanisverma yeah thats about right, thanks |
Thanks @bpasero :) I am thinking changing it to
Essentially I am thinking of swapping word "kept" to "pinned". if it sounds good, i will go ahead and create a PR for the same. |
@karanisverma yeah thats good. Also maybe fix that typo: |
Hey @bpasero Thanks a lot! |
Some users don't like the fact that editors are reused if they are not "pinned" (e.g. #61569).
I know how to pin an editor by double-clicking but I had a hard time to find the corresponding setting
enablePreview
because "preview" isn't something I can relate to "pinning".IMO "enablePreview" is a less-than-ideal name (but it is probably too late to change this).
I suggest to at least mention the concept "pinned" in the description of
enablePreview
.If I search for "pinned" in settings I find only this (which appears to be completely bogus):
/cc @roblourens
The text was updated successfully, but these errors were encountered: