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

Feat: Global Search Phase 8 and 9 #2108

Merged
merged 134 commits into from
Jul 13, 2023
Merged

Feat: Global Search Phase 8 and 9 #2108

merged 134 commits into from
Jul 13, 2023

Conversation

finnar-bin and others added 30 commits April 19, 2023 13:33
* task: added sortby filter

* task: rename sortby filter

* task: add user filter

* task: close menu on select

* task: add select highlight

* task: add date range to date filters

* task: set params for daterange filter

* task: set active filter text

* task: remove date filter

* task: pass active daterange value to daterange picker

* fix: fixed ts error
* task: added sortby filter

* task: rename sortby filter

* task: add user filter

* task: close menu on select

* task: add select highlight

* task: add date range to date filters

* task: set params for daterange filter

* task: set active filter text

* task: remove date filter

* task: pass active daterange value to daterange picker

* fix: fixed ts error

* - do not open a new tab when filters are added
- unpin specific tab when search queries are present
- show search keyword on tab name for the search page

* task: sort by function

* task: user filter

* task: background color update

* task: spacing fix

* task: date filter func

* task: updated result count

* task: fixed alignment

* task: added no results for filter
* task: added advanced search button

* task: added filter button

* chore: removed comment

* task: added advanced search modal

* task: fixed console errors

* task: fixed console error

* task: add form fields

* task: added tooltips
* task: added advanced search button

* task: added filter button

* chore: removed comment

* task: added advanced search modal

* task: fixed console errors

* task: fixed console error

* task: add form fields

* task: added tooltips

* task: created hook for advance search data

* task: moved search data to content search comp

* task: added clear all function

* task: wire user filter

* task: import types

* task: moved everything back into the advance search modal

* task: moved date config to a separate file & added daterange modal

* task: added date handling for single date picker

* task: daterange handler

* task: wired advance search page

* task: wired advance search function

* task: spacing and color update

* fix: fixed search button not enabling

* task: added cypress test scenarios

* task: search page cypress tests

* task: advance search modal cypress tests

* task: added data-cy

* task: updated cy get chaining

* fix: height fix for the counter

* task: added tooltips and changed date filter to modified at

* task: vqa updates

* task: adjusted border radius
* task: always open the dropdown on click

* task: add recent searches hook

* task: add recent searches to dropdown

* task: created separated component for global search items

* task: updated subheader style

* task: padding updates

* task: cleanup

* task: unfocus textfield when user selects an item

* task: always show user typed value

* task: removed comment

* task: wire remove recent search

* task: save a max of 50 recent search items

* task: when user keyword exists as a saved recent search, render it as a recent search

* task: hide suggestions when no user-typed keyword is present

* task: fixed incorrect icon

* task: remove manual JSON.parse/stringify

* task: removed deep cloning
* task: added resource type filter

* task: updated tooltip text

* task: add resource type filter component in search page
* task: created util to get content title

* task: changed how content result is being taken

* task: created filtered models hook

* task: show models in top suggestions

* task: moved getting icon to utils

* task: added model type keyword matching

* task: changed model search hook

* task: update comment

* chore: added hook description

* task: search models via user zuid

* task: allow searching models via user name

* task: changed how data is taken from rtk query

* task: show models in search page results

* task: render model list item

* task: wire resource type filter

* task: cleanup

* task: added icons to dropdown options
* task: created util to get content title

* task: changed how content result is being taken

* task: created filtered models hook

* task: show models in top suggestions

* task: moved getting icon to utils

* task: added model type keyword matching

* task: changed model search hook

* task: update comment

* chore: added hook description

* task: search models via user zuid

* task: allow searching models via user name

* task: changed how data is taken from rtk query

* task: show models in search page results

* task: render model list item

* task: wire resource type filter

* task: cleanup

* task: create rtk query for scripts and stylesheets

* task: add hook to filter code files

* task: include code files in top suggestions

* task: support view and snippet files for .html searches

* task: remove unneeded special handling for .html search

* task: add code to resource type filter dropdowns

* task: fixed type errors

* task: change type for resource types
@finnar-bin finnar-bin self-assigned this Jul 11, 2023
@finnar-bin finnar-bin requested a review from agalin920 July 11, 2023 04:56
@finnar-bin finnar-bin changed the title Feat/global search phase 8 Feat: Global Search Enhancements Jul 11, 2023
@finnar-bin
Copy link
Contributor Author

  1. If a user makes a blank query (" " or ""), do not add it as a recent search to the recent search list in the dropdown.
  2. If no user actions are found for a content item, model, or code file, then show the created at date.
  3. Sort the options in the Language dropdown in Alphabetical Order (A to Z)

This has been addressed

@finnar-bin finnar-bin requested a review from agalin920 July 13, 2023 01:56
@finnar-bin finnar-bin changed the title Feat: Global Search Enhancements Feat: Global Search Enhancements & Phase 9 Jul 13, 2023
@finnar-bin finnar-bin changed the title Feat: Global Search Enhancements & Phase 9 Feat: Global Search Phase 8 and 9 Jul 13, 2023
Copy link

@zcolah zcolah left a comment

Choose a reason for hiding this comment

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

Great job like always Nar.

  1. Can you ensure this is not case sensitive? So that it allows "in:content", etc.
  2. This is not part of the original spec but can you add the following keyboard interactions
  • When the user clicks on the down arrow key please allow them to also toggle between the accelerators (see how Slack does this)
  • When the user clicks on tab please allow them to toggle down to the next item, similar to what the down arrow key interaction does (see how Slack does this).

@shrunyan shrunyan merged commit 0977832 into master Jul 13, 2023
@shrunyan shrunyan deleted the feat/global-search-phase-8 branch July 13, 2023 22:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment