Skip to content

feat(search): replace search popover with inline search field - #2314

Merged
Antreesy merged 2 commits into
masterfrom
feat/inline-log-search
Oct 17, 2026
Merged

Antreesy merged 2 commits into
masterfrom
feat/inline-log-search

Conversation

@Antreesy

Copy link
Copy Markdown
Collaborator
  • Show the search input directly in the table header
  • Handle Ctrl+F via useHotKey from @nextcloud/vue
  • List the search shortcut in the app settings dialog
  • ⚠️ drop NcActionInput and date-fns compiled JS assets ⚠️
    • This is a thing in every app now. We should probably look into isolating datetimepicker into separate component somehow for it?
Before After
2026-10-17_13h41_01 2026-10-17_13h41_41
No 2026-10-17_14h05_38

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI (ClaudeCode:claude-opus-5-5)

- Show the search input directly in the table header
- Handle Ctrl+F via useHotKey from @nextcloud/vue
- List the search shortcut in the app settings dialog

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>

@susnux susnux left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✨

@susnux

susnux commented Oct 17, 2026

Copy link
Copy Markdown
Contributor

(not tested only from screenshot + code review 👍 )

@Antreesy

Copy link
Copy Markdown
Collaborator Author

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@Antreesy

Copy link
Copy Markdown
Collaborator Author

/backport 6b0369a to stable35

@Antreesy

Copy link
Copy Markdown
Collaborator Author

/backport 6b0369a to stable34

@Antreesy

Copy link
Copy Markdown
Collaborator Author

/backport 6b0369a to stable33

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants