Skip to content

fix: added code for citation and chat history panel independence #911

Merged
Avijit-Microsoft merged 10 commits into
devfrom
psl-panel-fix
May 14, 2026
Merged

fix: added code for citation and chat history panel independence #911
Avijit-Microsoft merged 10 commits into
devfrom
psl-panel-fix

Conversation

@Yamini-Microsoft
Copy link
Copy Markdown
Contributor

Purpose

This pull request makes several improvements to the deployment documentation and refactors the way the citation panel is displayed in the application UI. The most important changes include adding Bicep CLI as a required tool in the documentation, and updating the citation panel to render as a fixed overlay instead of within the main layout, simplifying its visibility logic.

Documentation updates:

  • Added Bicep CLI (v0.33.0+) as a required tool in both DeploymentGuide.md and AVMPostDeploymentGuide.md to ensure users have all necessary dependencies for deployment. [1] [2]

UI/Code refactoring:

  • Updated the citation panel to render as a fixed overlay using the new .citation-overlay CSS class, improving its visibility and separation from the main layout. [1] [2]
  • Removed outdated logic that set the citation panel’s width based on the chat history panel and instead always displays it as a fixed overlay.
  • Simplified the panel state handler by removing unnecessary dispatch of hideCitation, and updated dependencies for the related callback. [1] [2]

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

  • Citation panel appears as a fixed overlay when clicking a citation link
  • Citation panel stays visible when toggling chat history or dashboard
    panels
  • Dismiss (X) button closes the citation panel correctly
  • Panel layouts (Dashboard, Chat, History) are unaffected by the citation
    overlay
  • No UI overlap or clipping on resize

Other Information

Avijit-Microsoft and others added 9 commits April 20, 2026 10:59
fix: dev to main - Add location parameter to AI search service and update package versions, improvements to local development setup
Adds Bicep CLI (v0.33.0+) to the local deployment prerequisites so users provisioning this accelerator locally have the required Bicep version installed before running 'azd up' / 'az deployment'.

Work item: AB#42634
docs: Add Bicep CLI (v0.33.0+) to local deployment prerequisites
chore: dev to main merge
Remove unconditional hideCitation() call from onHandlePanelStates so
toggling one panel no longer closes the other. Citation is still
correctly dismissed on conversation switch and history clear.

Fixes #43310

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Move CitationPanel outside the flex layout container (.main-container)
and render it as a fixed-position overlay. This prevents the two panels
from competing for flex space, which caused:
- Closing Chat History to also close Citation panel
- Opening Citation to push Chat History off-screen

Fixes AB#43310

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
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 updates deployment prerequisites documentation and refactors the citation panel so it renders independently from the main dashboard/chat/history layout as a fixed overlay.

Changes:

  • Added Bicep CLI to deployment prerequisite documentation.
  • Moved citation panel rendering outside the main flex layout.
  • Added .citation-overlay styling and stopped hiding citations on panel toggle.

Reviewed changes

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

File Description
src/App/src/App.tsx Renders CitationPanel as a sibling overlay and updates panel toggle behavior.
src/App/src/App.css Adds fixed-position citation overlay styling.
documents/DeploymentGuide.md Adds Bicep CLI to local deployment required tools.
documents/AVMPostDeploymentGuide.md Adds Bicep CLI to post-deployment required tools.
Comments suppressed due to low confidence (1)

src/App/src/App.css:59

  • With min-width: 280px on a fixed panel and no small-screen override, the citation overlay becomes wider than the viewport on very narrow windows and will be clipped/off-screen. Use a responsive width cap (for example based on 100vw) so the dismiss button and content remain reachable during resize.
  width: 20%;
  min-width: 280px;
  max-width: 400px;

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

Comment thread src/App/src/App.css Outdated
Comment thread documents/AVMPostDeploymentGuide.md
Address review feedback: CitationPanel is now a flex participant
inside .main-container using .citation-panel-wrapper (flex: 0 0 20%)
instead of a fixed-position overlay. This reserves layout space and
avoids blocking clicks on underlying panels.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@Avijit-Microsoft Avijit-Microsoft merged commit 98d88ae into dev May 14, 2026
4 checks passed
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.

5 participants