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

Enhancements : improve most recent global search UI #41384

Merged
merged 3 commits into from
Nov 14, 2023

Conversation

nfebe
Copy link
Contributor

@nfebe nfebe commented Nov 10, 2023

  • Fixes remove person filter/chip when applied
  • Close person searchable filter via workaround

Resolves : #41381

TODO

  • Icon inversion which does not work well now since icons come in white or black instead of one consistent color. (Black icons turn white and white icons turn black, leading to to white on white background issues)
  • Use the new people filter (selector) that @marcoambrosini is working on (Current people filter not completed) - DONE, however, NcPopover which is used has issues with closing triggers.
  • Update back-end API (if ability not already present) to show "Search in App" dynamically.
  • Remove person filter not possible

From @ChristophWurst

  • BUG: Clicking outside the search modal needs two clicks on the magnifier icon to get the modal back.
  • BUG: Some app icons are white on white.
  • BUG: Modified timestamp bubbles change size on hover, have no pointing cursor.
  • BUG: Generally no feedback for no results.
  • BUG: Search people has white text on white background.
  • BUG: Search people has a tiny user bubble, barely wider than the avatar.
  • BUG: Search people No result goes to the top, overlaps other input.
  • BUG: Search people has no effect at all. The search string doesn't contain a person identifier.

@nfebe nfebe requested a review from marcoambrosini November 10, 2023 16:00
@AndyScherzinger AndyScherzinger added this to the Nextcloud 28 milestone Nov 10, 2023
@nfebe nfebe force-pushed the 41381-global-search-follow-up branch from e6032af to 97d5956 Compare November 10, 2023 17:54
@blizzz blizzz mentioned this pull request Nov 13, 2023
@marcoambrosini
Copy link
Member

marcoambrosini commented Nov 13, 2023

Design review notes

High

Medium

  • Contacts filter shouldn’t be available if contacts app is not available
  • No space for icons in modified
  • filter buttons justified to the left
  • Close icon shouldn't be jumping on hover

Low

  • Empty content vertically centered
  • Apply range primary to the right
  • Date range filter -> custom date range
  • Apply range -> Search in date range
  • 4px gap between all the elements in the input, filters, tags
  • Text in people regular not bold
  • Close icon material design icon "close" instead of "close-thick"
  • Cursor pointer instead on hovering close icon
  • Avatar in filter chips shouldn't be clickable and size should be 24px

@Altahrim
Copy link
Collaborator

Performance

  • /providers should be called only when user interact with the search, not on every page load

UI

  • "Modified" button should be renamed as we are not always searching on a modification date

@nfebe nfebe force-pushed the 41381-global-search-follow-up branch 5 times, most recently from 66f14b0 to abf4d32 Compare November 14, 2023 17:15
Copy link
Member

@AndyScherzinger AndyScherzinger left a comment

Choose a reason for hiding this comment

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

can't judge, but 👍 for merging the improvements for beta3 build

@nfebe
Copy link
Contributor Author

nfebe commented Nov 14, 2023

FYI : @blizzz and others, open items on this ticket would be moved to the a new PR and issue would open (or re-opened).

For beta3 merge considerations

@nfebe
Copy link
Contributor Author

nfebe commented Nov 14, 2023

/compile amend /

@nfebe nfebe requested review from artonge and juliusknorr November 14, 2023 17:26
- Fixes remove person filter/chip when applied
- Close person searchable filter via workaround
- Person filter has no effect fixed, now uses user property instead of id
- Left align filters
- Reduce avatar size in user chip and make photo non-clickable

Resolves : #41381

Signed-off-by: fenn-cs <[email protected]>
Show some feedback when there are no search results.

Signed-off-by: fenn-cs <[email protected]>
@nfebe nfebe force-pushed the 41381-global-search-follow-up branch from abf4d32 to a95396e Compare November 14, 2023 19:29
@nfebe
Copy link
Contributor Author

nfebe commented Nov 14, 2023

/compile amend /

Only show in-app serach button if available for specific app
 in question.

Signed-off-by: fenn-cs <[email protected]>
Signed-off-by: nextcloud-command <[email protected]>
@nextcloud-command nextcloud-command force-pushed the 41381-global-search-follow-up branch from a95396e to 987ac1e Compare November 14, 2023 20:07
@blizzz blizzz added 4. to release Ready to be released and/or waiting for tests to finish and removed 2. developing Work in progress labels Nov 14, 2023
@nfebe nfebe merged commit 230b1e3 into master Nov 14, 2023
@nfebe nfebe deleted the 41381-global-search-follow-up branch November 14, 2023 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish ❄️ 2023-Winter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Global search beta follow up
7 participants