feat: [OCISDEV-197] replace title only filter pill with toggle #12915
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
There was a problem hiding this comment.
Pull Request Overview
This PR replaces the "Title only" search filter pill with a toggle between "Title Only" and "Full Text Search" to improve user experience by making the search options clearer and more intuitive.
Key changes:
- Changed from a toggle button component to an inline filter component with two explicit options
- Added proper translation strings for both search modes using
$pgettext - Updated URL parameter handling to support the new filter structure
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
packages/web-app-search/src/portals/SearchBar.vue |
Added URL parameter handling for full text search state |
packages/web-app-files/src/components/Search/List.vue |
Replaced toggle component with inline filter and added search scope options |
changelog/unreleased/enhancement-use-switch-for-full-text-search-filter.md |
Added changelog entry documenting the enhancement |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
2bce3b7 to
9537f01
Compare
|
@mmattel we'll need to update the EN string for the "Title only" to change it into "Title Only". Have you got permissions in transifex to do EN translations? |
mzner
approved these changes
Aug 14, 2025
We've replaced the "Title only" search filter pill with a toggle between "Title Only" and "Full Text Search". This change should improve the user experience as it's easier to understand the toggle.
9537f01 to
cdab204
Compare
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
We've replaced the "Title only" search filter pill with a toggle between "Title Only" and "Full Text Search". This change should improve the user experience as it's easier to understand the toggle.
Motivation and Context
Better UX.
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes