Skip to content

Conversation

@JammingBen
Copy link
Contributor

Adds the tiles view to the search result page and refactors the component to script setup.

closes #1351

@JammingBen JammingBen self-assigned this Oct 17, 2025
Copilot AI review requested due to automatic review settings October 17, 2025 09:51
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a tiles view option to the search result page and refactors the List component from Options API to Composition API using script setup. The implementation enables users to switch between different view modes (table and tiles) when viewing search results.

  • Adds support for folder view extensions in the search context
  • Refactors the Search List component from Options API to script setup syntax
  • Enables tiles view functionality for search results with additional content slots

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
packages/web-pkg/src/components/FilesList/ResourceTiles.vue Adds slot for additional resource content in tiles view
packages/web-pkg/src/components/FilesList/ResourceTile.vue Adds slot for additional content in individual tile component
packages/web-app-files/tests/unit/components/Search/List.spec.ts Updates tests for refactored component and new extension system
packages/web-app-files/src/extensionPoints.ts Adds new extension point for search folder views
packages/web-app-files/src/composables/extensions/useFolderViews.ts Registers search extension point for all folder view types
packages/web-app-files/src/components/Search/List.vue Complete refactor to script setup with tiles view support

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

@AlexAndBear AlexAndBear left a comment

Choose a reason for hiding this comment

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

🤩

@JammingBen JammingBen merged commit 7b2907b into main Oct 17, 2025
29 checks passed
@JammingBen JammingBen deleted the feat/search-result-page-tiles-view branch October 17, 2025 11:30
@openclouders openclouders mentioned this pull request Oct 17, 2025
1 task
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.

Tiles view: Search result page

3 participants