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

Fenn-CS
Copy link
Contributor

@Fenn-CS Fenn-CS 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.

@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

@Fenn-CS Fenn-CS 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

@Fenn-CS
Copy link
Contributor Author

Fenn-CS 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

@Fenn-CS
Copy link
Contributor Author

Fenn-CS commented Nov 14, 2023

/compile amend /

- 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 <fenn25.fn@gmail.com>
Show some feedback when there are no search results.

Signed-off-by: fenn-cs <fenn25.fn@gmail.com>
@Fenn-CS Fenn-CS force-pushed the 41381-global-search-follow-up branch from abf4d32 to a95396e Compare November 14, 2023 19:29
@Fenn-CS
Copy link
Contributor Author

Fenn-CS commented Nov 14, 2023

/compile amend /

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

Signed-off-by: fenn-cs <fenn25.fn@gmail.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@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
@Fenn-CS Fenn-CS merged commit 230b1e3 into master Nov 14, 2023
39 of 42 checks passed
@Fenn-CS Fenn-CS 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