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

fix: Export Setup Components Filter Negation #2643

Merged
merged 1 commit into from
Jul 28, 2024

Conversation

phroggster
Copy link
Collaborator

Description of the Change

  • Fix the Export Setup Component dialog filtering.
    • Angular will negate filters that start with an exclamation mark, which is cool. However, many commands start with an exclamation mark, so that negation behavior is undesired here.
    • This completely removes any leading exclamation marks from the filtering.
  • Cleaned up a trivial arrow-parens lint warning.

Applicable Issues

None applicable

Testing

Tried a number of things to break it, but I think even this one will get past CK.

Screenshots

Screenshot showing searching for (in clockwise order):

  • Nothing
  • Previously-negated text
  • Group/Sorting tags
  • Identifiers

command searching round 2

@phroggster phroggster requested a review from ebiggz as a code owner July 28, 2024 16:13
@phroggster phroggster force-pushed the fix_setup_filtering_negation branch from bccc199 to d13f798 Compare July 28, 2024 16:17
Angular will negate filters that start with an exclamation mark, which
is cool. However, many commands start with an exclamation mark, so the
negation behavior is undesired here.
Also cleaned up a trivial arrow-parens lint warning.
@phroggster phroggster force-pushed the fix_setup_filtering_negation branch from d13f798 to f9ef09b Compare July 28, 2024 17:23
@phroggster
Copy link
Collaborator Author

phroggster commented Jul 28, 2024

I love how I can rebase a PR (edit: on the web) to get it up to the HEAD of the base, but can't pull that commit back down locally. Sigh.

@ebiggz ebiggz merged commit 2bdc664 into crowbartools:v5 Jul 28, 2024
1 check passed
@phroggster phroggster deleted the fix_setup_filtering_negation branch July 31, 2024 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants