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

Add option for swiper line number to be searchable #562

Closed
wants to merge 1 commit into from

Conversation

@justbur
Copy link
Contributor

@justbur justbur commented Jun 17, 2016

  • swiper.el (swiper-include-line-number-in-search): Add option
  • swiper.el: Use swiper-line-number to store candidate line number
  • ivy.el: Same

Store swiper line number in new text property swiper-line-number,
reserving the display property for the display of text. When the option
is enabled the only change is that the line number is included as text
in the candidates instead of in the display property.

 * swiper.el (swiper-include-line-number-in-search): Add option
 * swiper.el: Use swiper-line-number to store candidate line number
 * ivy.el: Same

Store swiper line number in new text property swiper-line-number,
reserving the display property for the display of text. When the option
is enabled the only change is that the line number is included as text
in the candidates instead of in the display property.
@abo-abo abo-abo closed this in e89f250 Jun 17, 2016
@abo-abo
Copy link
Owner

@abo-abo abo-abo commented Jun 17, 2016

Thanks. Funny thing, the line numbers were originally part of the candidates. Excluding them was a later feature. Doesn't hurt to have the option though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants