Skip to content

Conversation

@nsdeschenes
Copy link
Contributor

@nsdeschenes nsdeschenes commented Nov 17, 2025

Currently the logic category only renders when the filter item is not the first item in the search bar. This PR tweaks this to render the category, but with only the opening paren item, as it is the only valid one in that location.

This PR removes the first item checks, and will display all logic operators at all times.

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Nov 17, 2025
@nsdeschenes nsdeschenes marked this pull request as ready for review November 18, 2025 11:34
Copy link
Member

@malwilley malwilley left a comment

Choose a reason for hiding this comment

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

I know it's not exactly valid, but IMO it would be a better experience if we just show all options all the time, so users understand what they are able to do

@nsdeschenes nsdeschenes force-pushed the nd/feat-search-bar-tweak-logic-category-conditions branch from ca8284b to 34fc3cd Compare November 19, 2025 15:47
@nsdeschenes nsdeschenes merged commit 498dbfc into master Nov 19, 2025
49 checks passed
@nsdeschenes nsdeschenes deleted the nd/feat-search-bar-tweak-logic-category-conditions branch November 19, 2025 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants