Skip to content

Conversation

@ArtyomSavchenko
Copy link
Member

@ArtyomSavchenko ArtyomSavchenko commented Sep 25, 2025

Screenshot 2025-09-25 at 11 49 24 Screenshot 2025-09-25 at 12 06 23

Signed-off-by: Artem Savchenko <armisav@gmail.com>
Signed-off-by: Artem Savchenko <armisav@gmail.com>
@huly-github-staging
Copy link

Connected to Huly®: UBERF-13771

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 an "All Cards" section to the card navigation interface and makes the feed view the default view for cards. The changes introduce a new "all" selection type that displays all cards in the system using a centralized feed view.

  • Added "All Cards" special navigation item with proper localization
  • Set feed view as the default viewlet for card display
  • Modified selection logic to default to "all" view when no specific card is selected

Reviewed Changes

Copilot reviewed 16 out of 17 changed files in this pull request and generated no comments.

Show a summary per file
File Description
plugins/chat-resources/src/components/ChatApplication.svelte Sets default selection to "all" type when no card is selected
plugins/card/src/index.ts Adds "All" icon and "AllCards" string resources
plugins/card-resources/src/components/Main.svelte Sets CardFeedDescriptor as the default viewlet
plugins/card-resources/src/components/CardFeedView.svelte Conditionally passes card type to NewCardForm based on class
plugins/card-assets/src/index.ts Maps "All" icon to the all symbol
plugins/card-assets/lang/*.json Adds "AllCards" translations in 9 languages
models/card/src/index.ts Adds "all" special navigation item with feed view configuration

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

@ArtyomSavchenko ArtyomSavchenko marked this pull request as ready for review September 25, 2025 05:13
@haiodo haiodo merged commit 7ea518f into develop Sep 25, 2025
15 checks passed
@haiodo haiodo deleted the cards-all branch September 25, 2025 05:59
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.

3 participants