Skip to content

🐛 Bugfix: Fix incomplete display of tenant resources page after window resize#3215

Open
xuyaqist wants to merge 3 commits into
developfrom
xyq/refactor_chat
Open

🐛 Bugfix: Fix incomplete display of tenant resources page after window resize#3215
xuyaqist wants to merge 3 commits into
developfrom
xyq/refactor_chat

Conversation

@xuyaqist

@xuyaqist xuyaqist commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

#3214 Bugfix: Fix incomplete display of tenant resources page after window resize

Copilot AI review requested due to automatic review settings June 10, 2026 08:05
@xuyaqist xuyaqist requested review from Dallas98 and WMC001 as code owners June 10, 2026 08:05
@xuyaqist xuyaqist changed the title Move non-shadcn ui component to other folder ♻️ Move non-shadcn ui component to other folder Jun 10, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 reorganizes frontend components by moving non-shadcn UI components out of components/ui into components/common, and updates all consuming imports accordingly. This helps keep components/ui reserved for shadcn-style primitives while housing app-specific widgets/renderers under common.

Changes:

  • Added non-shadcn components under frontend/components/common (e.g., PdfViewer, Diagram, CopyButton, TokenUsageIndicator).
  • Updated imports across chat/agents/knowledge pages to reference @/components/common/* instead of @/components/ui/*.
  • Updated internal common components (e.g., markdownRenderer, filePreviewDrawer) to reference the relocated components.

Reviewed changes

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

Show a summary per file
File Description
frontend/components/common/tokenUsageIndicator.tsx Adds token usage ring + tooltip as a common component.
frontend/components/common/PdfViewer.tsx Adds PDF viewer as a common component and wires it for file preview usage.
frontend/components/common/Diagram.tsx Adds Mermaid diagram renderer as a common component.
frontend/components/common/copyButton.tsx Adds reusable copy-to-clipboard button as a common component.
frontend/components/common/markdownRenderer.tsx Updates imports to use common CopyButton/Diagram.
frontend/components/common/filePreviewDrawer.tsx Updates imports/dynamic import to use common markdown renderer and PdfViewer.
frontend/app/[locale]/tenant-resources/components/resources/KnowledgeList.tsx Updates MarkdownRenderer import to common.
frontend/app/[locale]/knowledges/components/document/DocumentList.tsx Updates MarkdownRenderer/FilePreviewDrawer imports to common.
frontend/app/[locale]/chat/streaming/taskWindow.tsx Updates MarkdownRenderer/CodeBlock import to common.
frontend/app/[locale]/chat/streaming/chatStreamFinalMessage.tsx Updates MarkdownRenderer import to common.
frontend/app/[locale]/chat/components/chatInput.tsx Updates FilePreviewDrawer/TokenUsageIndicator imports to common.
frontend/app/[locale]/chat/components/chatAttachment.tsx Updates FilePreviewDrawer import to common.
frontend/app/[locale]/agents/components/agentConfig/SkillDetailModal.tsx Updates MarkdownRenderer import to common.
frontend/app/[locale]/agents/components/agentConfig/SkillBuildModal.tsx Updates MarkdownRenderer import to common.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@xuyaqist xuyaqist changed the title ♻️ Move non-shadcn ui component to other folder Bugfix: Fix incomplete display of tenant resources page after window resize Jun 11, 2026
@xuyaqist xuyaqist changed the title Bugfix: Fix incomplete display of tenant resources page after window resize 🐛 Bugfix: Fix incomplete display of tenant resources page after window resize Jun 11, 2026
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.

2 participants