Skip to content

Highlight match letters#8868

Closed
donkawechico wants to merge 7 commits intodevfrom
highlight-match-letters
Closed

Highlight match letters#8868
donkawechico wants to merge 7 commits intodevfrom
highlight-match-letters

Conversation

@donkawechico
Copy link
Copy Markdown
Contributor

@donkawechico donkawechico commented Apr 8, 2021

Proposed change

Highlight the letters found with VS Code's fuzzy sequence matching algorithm (see screenshots).

Command Items

Entity Items

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (thank you!)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Example configuration

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue or discussion:
  • Link to documentation pull request:

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

@donkawechico donkawechico added the Do Not Review PR is not ready for any kind of review label Apr 8, 2021
@donkawechico donkawechico force-pushed the highlight-match-letters branch from 11aa903 to f513703 Compare April 8, 2021 23:40
Comment thread src/common/string/filter/sequence-matching.ts Outdated
Comment thread src/common/string/filter/sequence-matching.ts Outdated
@bramkragten
Copy link
Copy Markdown
Member

Btw, what does that do not review badge mean? 😅

@donkawechico donkawechico force-pushed the highlight-match-letters branch from f513703 to c05e566 Compare April 15, 2021 02:31
@donkawechico
Copy link
Copy Markdown
Contributor Author

donkawechico commented Apr 15, 2021

Btw, what does that do not review badge mean? 😅

I forget the exact context, but I made that ages ago cuz you kept reviewing my draft PRs! 😜

I tend to do a lot of early iterating in drafts and I don't want to waste folks' time.

@donkawechico donkawechico force-pushed the highlight-match-letters branch from c05e566 to 3163eec Compare April 15, 2021 03:58
@donkawechico donkawechico removed the Do Not Review PR is not ready for any kind of review label Apr 15, 2021
@donkawechico donkawechico marked this pull request as ready for review April 20, 2021 18:33
Comment thread src/common/string/filter/sequence-matching.ts Outdated
Comment thread src/common/string/filter/sequence-matching.ts Outdated
Comment thread src/common/string/filter/sequence-matching.ts Outdated
Comment thread src/dialogs/quick-bar/ha-quick-bar.ts Outdated
@donkawechico donkawechico force-pushed the highlight-match-letters branch from d1b512e to fdf1eae Compare April 28, 2021 22:19
@donkawechico donkawechico force-pushed the highlight-match-letters branch 3 times, most recently from 9fb8ba1 to 72a2c98 Compare April 29, 2021 00:22
@donkawechico donkawechico force-pushed the highlight-match-letters branch from 72a2c98 to 448f5d7 Compare April 29, 2021 00:23
score?: number;
strings: string[];
decoratedStrings?: Decoration[][][];
treatArrayAsSingleString?: boolean;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we move this to an options object param on FuzzyFilterSort?

@github-actions
Copy link
Copy Markdown

There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days.
Thank you for your contributions.

@github-actions github-actions Bot added the stale label Jul 28, 2021
@github-actions github-actions Bot closed this Aug 4, 2021
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants