Skip to content

Conversation

@agatha197
Copy link
Contributor

@agatha197 agatha197 commented Jan 7, 2026

Resolves #4960 (FR-1878)

Enhance Folder Explorer with Project Name Display and Alert Type Change

This PR enhances the folder explorer modal to display the specific project name when viewing folders from different projects. Previously, users would only see a generic warning message stating "This folder belongs to a different project." Now, the message includes the actual project name: "This folder belongs to '[project name]' project."

Key changes:

  • Added group_name to the GraphQL query to fetch the project name
  • Changed the alert type from "warning" to "info" for a less alarming visual appearance
  • Updated the message template in all language files to include the project name parameter
  • Added missing translation for "View in folder explorer" in several languages

These changes improve the user experience by providing more specific information about folder ownership, making it easier for users to understand which project a folder belongs to when navigating across different projects.

image.png

Copilot AI review requested due to automatic review settings January 7, 2026 05:37
@github-actions github-actions bot added area:ux UI / UX issue. area:i18n Localization size:L 100~500 LoC labels Jan 7, 2026
Copy link
Contributor Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more


How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • flow:merge-queue - adds this PR to the back of the merge queue
  • flow:hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has required the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 7, 2026

Coverage report for ./react

St.
Category Percentage Covered / Total
🔴 Statements 4.91% 589/11992
🔴 Branches
4.47% (-0% 🔻)
377/8433
🔴 Functions 2.75% 101/3676
🔴 Lines 4.74% 555/11711

Test suite run success

173 tests passing in 13 suites.

Report generated by 🧪jest coverage report action from 2f4a4d3

@agatha197 agatha197 force-pushed the feat_FR-1878_/add-switch-project-button branch from 27db591 to d1a9bca Compare January 7, 2026 05:42
@agatha197 agatha197 force-pushed the feat_FR-1871_/add-direct-access-to-model-card-modal-header branch from b76d2d3 to 0b2f1eb Compare January 7, 2026 05:42
@agatha197 agatha197 review requested due to automatic review settings January 7, 2026 05:44
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 "Switch Project" button to the folder explorer when viewing a folder that belongs to a different project than the currently active one. Previously, users would only see a warning message without the ability to easily switch projects, requiring them to manually navigate away and change projects through other UI elements.

Key changes:

  • Created a new FolderProjectMismatchAlert component that displays the warning with an actionable "Switch Project" button
  • Added comprehensive translations for the new button and messages across all 21 supported languages
  • Added a tooltip to the folder view icon in the model card modal for better discoverability

Reviewed changes

Copilot reviewed 24 out of 24 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
react/src/components/FolderProjectMismatchAlert.tsx New component that displays a project mismatch alert with a "Switch Project" button
react/src/components/FolderExplorerModal.tsx Updated to use the new FolderProjectMismatchAlert component instead of a simple alert
react/src/components/ModelCardModal.tsx Added tooltip to the folder view icon for better UX
resources/i18n/en.json Added English translations for "SwitchProject", "SwitchedToProject", "FailedToSwitchProject", and "ViewInFolderExplorer"
resources/i18n/ko.json Added Korean translations
resources/i18n/ja.json Added Japanese translations
resources/i18n/zh-CN.json Added Simplified Chinese translations
resources/i18n/zh-TW.json Added Traditional Chinese translations
resources/i18n/de.json Added German translations
resources/i18n/es.json Added Spanish translations
resources/i18n/fr.json Added French translations
resources/i18n/it.json Added Italian translations
resources/i18n/pt.json Added Portuguese translations
resources/i18n/pt-BR.json Added Brazilian Portuguese translations
resources/i18n/ru.json Added Russian translations
resources/i18n/pl.json Added Polish translations
resources/i18n/tr.json Added Turkish translations
resources/i18n/vi.json Added Vietnamese translations
resources/i18n/th.json Added Thai translations
resources/i18n/ms.json Added Malay translations
resources/i18n/mn.json Added Mongolian translations
resources/i18n/el.json Added Greek translations
resources/i18n/fi.json Added Finnish translations
resources/i18n/id.json Added Indonesian translations

Copilot AI review requested due to automatic review settings January 7, 2026 05:48
@agatha197 agatha197 force-pushed the feat_FR-1878_/add-switch-project-button branch from 2175d6b to e2ec60b Compare January 7, 2026 05:48
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

Copilot reviewed 24 out of 24 changed files in this pull request and generated 1 comment.

@agatha197 agatha197 marked this pull request as draft January 7, 2026 07:42
@agatha197 agatha197 changed the base branch from feat_FR-1871_/add-direct-access-to-model-card-modal-header to graphite-base/4961 January 7, 2026 08:25
Copilot AI review requested due to automatic review settings January 7, 2026 08:38
@agatha197 agatha197 force-pushed the feat_FR-1878_/add-switch-project-button branch from 3f3d674 to 538ba51 Compare January 7, 2026 08:38
@agatha197 agatha197 changed the base branch from graphite-base/4961 to feat_FR-1871_/add-direct-access-to-model-card-modal-header January 7, 2026 08:39
@agatha197 agatha197 changed the title feat(FR-1878): Add a button to change the project in the folder explorer feat(FR-1878): Show project name in folder explorer for cross-project folders Jan 7, 2026
@agatha197 agatha197 marked this pull request as ready for review January 7, 2026 08:39
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

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

@agatha197 agatha197 force-pushed the feat_FR-1878_/add-switch-project-button branch from 538ba51 to d67125b Compare January 7, 2026 08:44
@github-actions github-actions bot added size:M 30~100 LoC and removed size:L 100~500 LoC labels Jan 7, 2026
@agatha197 agatha197 changed the base branch from feat_FR-1871_/add-direct-access-to-model-card-modal-header to graphite-base/4961 January 9, 2026 07:01
Copilot AI review requested due to automatic review settings January 9, 2026 07:01
@agatha197 agatha197 force-pushed the feat_FR-1878_/add-switch-project-button branch from d67125b to 553c366 Compare January 9, 2026 07:01
@agatha197 agatha197 changed the base branch from graphite-base/4961 to feat_FR-1871_/add-direct-access-to-model-card-modal-header January 9, 2026 07:01
@agatha197 agatha197 requested a review from ironAiken2 January 9, 2026 07:02
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

Copilot reviewed 21 out of 21 changed files in this pull request and generated 1 comment.

Copy link
Contributor

@ironAiken2 ironAiken2 left a comment

Choose a reason for hiding this comment

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

Copy link
Contributor

@ironAiken2 ironAiken2 left a comment

Choose a reason for hiding this comment

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

please check copilot's review. It appears that the translation file is missing.

@agatha197 agatha197 force-pushed the feat_FR-1871_/add-direct-access-to-model-card-modal-header branch from a264738 to b07f323 Compare January 9, 2026 07:53
@agatha197 agatha197 force-pushed the feat_FR-1878_/add-switch-project-button branch from 553c366 to 2f4a4d3 Compare January 9, 2026 07:53
Copy link
Contributor Author

please check copilot's review. It appears that the translation file is missing.

;; my extension is strange. I applied it.

@agatha197 agatha197 requested a review from ironAiken2 January 9, 2026 07:53
Copy link
Contributor

@ironAiken2 ironAiken2 left a comment

Choose a reason for hiding this comment

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

It seems we need to modify NotInProject to accept variable assignment :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:i18n Localization area:ux UI / UX issue. size:M 30~100 LoC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants