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(3000): more search improvements #18759

Merged
merged 24 commits into from
Nov 22, 2023
Merged

feat(3000): more search improvements #18759

merged 24 commits into from
Nov 22, 2023

Conversation

thmsobrmlr
Copy link
Contributor

@thmsobrmlr thmsobrmlr commented Nov 20, 2023

Problem

The search command wasn't introduced to the user & various QOL issues.

Changes

This PR:

  • adds the search to the navbar
  • adds hints for cmd+shift+k to the search bar
  • adds tab behaviour to tabs
  • prevents going into search mode while pressing backspace continously (key repeat instead of dedicated backspace press)
  • improves styles

How did you test this code?

Clicking and typing

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

2 snapshot changes in total. 0 added, 2 modified, 0 deleted:

  • chromium: 0 added, 2 modified, 0 deleted (diff for shard 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

thmsobrmlr and others added 4 commits November 20, 2023 19:40
# Conflicts:
#	frontend/src/lib/components/CommandBar/ActionInput.tsx
#	frontend/src/lib/components/CommandBar/SearchInput.tsx
@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

1 snapshot changes in total. 0 added, 1 modified, 0 deleted:

  • chromium: 0 added, 1 modified, 0 deleted (diff for shard 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

4 snapshot changes in total. 0 added, 4 modified, 0 deleted:

  • chromium: 0 added, 4 modified, 0 deleted (diff for shard 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@thmsobrmlr thmsobrmlr force-pushed the search-improvements-3 branch from 23102a0 to 33a45ac Compare November 21, 2023 16:58
@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

4 snapshot changes in total. 0 added, 4 modified, 0 deleted:

  • chromium: 0 added, 4 modified, 0 deleted (diff for shard 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

Copy link
Contributor

github-actions bot commented Nov 21, 2023

Size Change: 0 B

Total Size: 1.97 MB

ℹ️ View Unchanged
Filename Size
frontend/dist/toolbar.js 1.97 MB

compressed-size-action

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

3 snapshot changes in total. 0 added, 3 modified, 0 deleted:

  • chromium: 0 added, 3 modified, 0 deleted (diff for shard 2)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@thmsobrmlr thmsobrmlr marked this pull request as ready for review November 22, 2023 09:45
@thmsobrmlr thmsobrmlr requested a review from a team November 22, 2023 10:10
@thmsobrmlr thmsobrmlr merged commit d78450e into master Nov 22, 2023
77 checks passed
@thmsobrmlr thmsobrmlr deleted the search-improvements-3 branch November 22, 2023 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants