Skip to content

Conversation

@ArtyomSavchenko
Copy link
Member

@ArtyomSavchenko ArtyomSavchenko commented Oct 22, 2025

Screenshot 2025-10-22 at 21 15 52

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

Connected to Huly®: UBERF-13973

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 functionality to favorite card types in the navigator. Users can now mark card types (MasterTags) as favorites, which are displayed in a separate "Favorites" section above the main types list.

Key changes:

  • Added FavoriteType model for storing favorite card type preferences
  • Enhanced TypesNavigator to display favorites section with star/unstar actions
  • Added internationalization support for "Favorites" label across 11 languages

Reviewed Changes

Copilot reviewed 15 out of 15 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
plugins/card/src/index.ts Added FavoriteType interface and class definition, plus Favorites string
plugins/card-resources/src/components/navigator/TypesNavigator.svelte Implemented favorite types logic, queries, and split UI into favorites/non-favorites sections
plugins/card-resources/src/components/navigator/TagHierarchy.svelte Added action button support and excluded classes filtering for hiding favorited items
plugins/card-assets/lang/*.json Added "Favorites" translations in 11 languages
models/card/src/index.ts Registered TFavoriteType model class

@ArtyomSavchenko ArtyomSavchenko marked this pull request as ready for review October 22, 2025 14:45
@haiodo haiodo merged commit 7875691 into develop Oct 23, 2025
15 checks passed
@haiodo haiodo deleted the favorite-types branch October 23, 2025 02:04
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