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

New anchored filter method #70

Merged
merged 1 commit into from
Oct 17, 2020

Conversation

ruliana
Copy link
Contributor

@ruliana ruliana commented Oct 5, 2020

New anchored filter method

The proposed filter uses uppercased letters and symbols as anchors
for word beginning.

For example, assuming the text "find-file-at-point":

  • Query "FFP" matches "(f)ind-(f)ile-at-(p)oint"
  • Query "FiPoint" matches "(fi)nd-file-at-(point)"
  • Query "fiFAP" matches "(fi)nd-(f)ile-(a)t-(p)oint"
  • Query "fi-a-p" matches "(fi)nd-file-(a)t-(p)oint"

The rationale for a new filter is that it has the convenience of initialism,
with more control to the user, but without increase the number of
keystrokes.

Copy link
Member

@raxod502 raxod502 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems good to me! Thank you for helping out :)
Just add a changelog entry and I'll be happy to merge.

README.md Outdated Show resolved Hide resolved
@ruliana
Copy link
Contributor Author

ruliana commented Oct 7, 2020

Seems good to me! Thank you for helping out :)
Just add a changelog entry and I'll be happy to merge.

Should I squash the commits? (Assuming "yes", but it's better to ask) 😬

"Anchored" uses uppercase letters and symbols as anchors for word
beginning.

For example, assuming the text "find-file-at-point":
- Query "FFP" matches "(find)-(file)-at-(point)"
- Query "FiPoint" matches "(find)-file-at-(point)"
- Query "fiFAP" matches "(find)-(file)-(at)-(point)"
- Query "fi-a-p" matches "(find)-file-(at)-(point)"

The rationale for a new filter is that it has the convenience of
"initialism", with more control, but without increase the number
of keystrokes.
@ruliana
Copy link
Contributor Author

ruliana commented Oct 8, 2020

All changes done 🙂

Copy link
Member

@raxod502 raxod502 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! Thank you!

@raxod502 raxod502 merged commit 47fef02 into radian-software:master Oct 17, 2020
@raxod502
Copy link
Member

Whether you squash or not is up to you. GitHub lets me do that automatically when I merge the pull request.

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

Successfully merging this pull request may close these issues.

2 participants