Skip to content

Conversation

@ArtyomSavchenko
Copy link
Member

@ArtyomSavchenko ArtyomSavchenko commented Oct 2, 2025

Screenshot 2025-10-02 at 18 04 11

Signed-off-by: Artem Savchenko <armisav@gmail.com>
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 enhances the card parent display functionality by populating parent information when attaching cards to messages and replacing the navigator component with a dedicated parent names presenter.

  • Updates card attachment to include parent card information (ID, class, and title) in the parentInfo array
  • Replaces ParentsNavigator with ParentNamesPresenter component for better parent display
  • Increases maximum width for parent display from 10rem to 12rem

Reviewed Changes

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

File Description
plugins/communication-resources/src/actions.ts Populates parentInfo array with parent card details when attaching cards
plugins/card-resources/src/components/EditCardNew.svelte Switches to ParentNamesPresenter component and adjusts display width

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

@huly-github-staging
Copy link

Connected to Huly®: UBERF-13846

@ArtyomSavchenko ArtyomSavchenko merged commit 082e36e into develop Oct 2, 2025
15 checks passed
Copy link
Contributor

haiodo commented Oct 2, 2025

will it be visible also on right panel?

@ArtyomSavchenko
Copy link
Member Author

will it be visible also on right panel?

For now it is visible only if panel width is large enough
Looks like we should display parent or back button in compact way in this case

@BykhovDenis BykhovDenis deleted the fix-parent-navigation branch October 22, 2025 18:25
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.

4 participants