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

What is the intent of TextSearchPreviewOptions.maxLines? #57419

Closed
bpasero opened this issue Aug 28, 2018 · 5 comments
Closed

What is the intent of TextSearchPreviewOptions.maxLines? #57419

bpasero opened this issue Aug 28, 2018 · 5 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug search Search widget and operation issues verified Verification succeeded
Milestone

Comments

@bpasero
Copy link
Member

bpasero commented Aug 28, 2018

Refs: #57305

I tried changing the TextSearchPreviewOptions.maxLines setting but could never see more than one line showing up. How to use it?

@bpasero bpasero changed the title What is the intent of maxLines? What is the intent of TextSearchPreviewOptions.maxLines? Aug 28, 2018
@roblourens
Copy link
Member

roblourens commented Aug 28, 2018

Multiline search isn't supported yet by the builtin search provider. (I should add a comment to that effect). When it is, this would let consumers request lines in the preview. It won't change the number of actual lines that the match can contain, just text lines in the preview.

For example, vscode will just display the first line in the search viewlet. But, an extension that shows results in an editor might want to show the first 5 lines of a match.

@bpasero
Copy link
Member Author

bpasero commented Aug 28, 2018

Ah, so this travels potentially to an extension that provides search, but is currently not supported by our built in search?

@roblourens
Copy link
Member

Correct

@roblourens
Copy link
Member

The builtin provider will probably support it in a month or two.

@bpasero
Copy link
Member Author

bpasero commented Aug 28, 2018

Maybe this should be documented so that it does not draw the wrong assumptions.

@roblourens roblourens added bug Issue identified by VS Code Team member as probable bug search Search widget and operation issues labels Aug 28, 2018
@roblourens roblourens added this to the August 2018 milestone Aug 28, 2018
@bpasero bpasero added the verified Verification succeeded label Aug 29, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Oct 12, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug search Search widget and operation issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

2 participants