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

[next] feat(NcAppNavigation): Provide consistent in-app search #5848

Merged
merged 1 commit into from
Jan 19, 2025

Conversation

backportbot[bot]
Copy link

@backportbot backportbot bot commented Jul 22, 2024

Backport of #5831

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot backportbot bot added enhancement New feature or request 3. to review Waiting for reviews feature: app-navigation Related to the app-navigation component labels Jul 22, 2024
@raimund-schluessler raimund-schluessler added this to the 9.0.0-alpha.6 milestone Aug 5, 2024
@ShGKme
Copy link
Contributor

ShGKme commented Nov 14, 2024

@susnux ready for review?

`NcAppNaviation` now provides an optional in-app search when `show-search` is set.
This allows apps which have in app filtering / search to use a consistent layout.

There is also an actions slot to provide some inline actions like filters.

Squashed: Make the `NcAppNavigationSearch` a fully separated component

Co-authored-by: Grigorii K. Shartsev <me@shgk.me>
Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux force-pushed the backport/5831/next branch from 24fe833 to c0a1cfe Compare January 19, 2025 13:24
@susnux
Copy link
Contributor

susnux commented Jan 19, 2025

Ready to review - visual artifacts as this backport is still missing: #5832

@susnux susnux marked this pull request as ready for review January 19, 2025 13:25
@susnux susnux merged commit 8d32524 into next Jan 19, 2025
21 checks passed
@susnux susnux deleted the backport/5831/next branch January 19, 2025 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews enhancement New feature or request feature: app-navigation Related to the app-navigation component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants