Skip to content

Fix responsive V2 chat table overflow - #1913

Merged
Paul Lizer (paullizer) merged 1 commit into
microsoft:Developmentfrom
paullizer:paullizer-responsive-chat-tables
Nov 10, 2026
Merged

Paul Lizer (paullizer) merged 1 commit into
microsoft:Developmentfrom
paullizer:paullizer-responsive-chat-tables

Conversation

@paullizer

Copy link
Copy Markdown
Collaborator

Summary

  • Wide Markdown evidence tables could extend past their assistant bubble. Table-bearing V2 replies now fill the available conversation container, while normal prose and user-message widths remain unchanged.
  • Long identifiers, JSON, and URLs wrap without reducing font size. A responsive minimum cell width avoids excessively narrow columns; tables that still cannot fit scroll inside a keyboard-focusable region instead of overflowing the conversation.
  • Saved, streaming, finalized, and reloaded replies share the layout. Classic chat and the comfortable/wide preference are unchanged.
  • Add functional/browser regressions and fix documentation; bump the application to 0.261.425.

Linked issue

N/A. The user requested proceeding without a GitHub issue.

Release Notes & Latest Features

  • New Feature
  • Bug Fix
  • UI Enhancement
  • Breaking Change
  • Internal only

Is this visible to end users?

  • Yes
  • No

Is this admin-facing (Admin Settings, governance, deployment, config)?

  • Yes
  • No

Should this become a Latest Feature card?

  • Yes
  • No
  • Already added

Screenshot needed for the card?

  • Yes
  • No
  • Attached

Version bump

  • application/single_app/config.py VERSION third segment bumped, or not needed because this is docs-only
  • deployers/version.txt bumped, or not needed because deployers/ was not changed

Application: 0.261.424 -> 0.261.425. No deployer changes.

Testing / validation

  • npm --prefix .\application\v2_ui run build -- --outDir ..\..\ui_tests\artifacts\orchestration-plan-editor - passed TypeScript compilation and production build; Vite reported its existing large-chunk warning.
  • python .\functional_tests\test_v2_chat_table_layout.py - 3 passed.
  • python .\functional_tests\test_v2_mobile_layout.py - 5 passed.
  • python -m pytest .\ui_tests\test_v2_chat_table_layout.py .\ui_tests\test_v2_mobile_chat_layout.py -q --tb=short - 18 passed (9 table regressions and 9 existing mobile regressions). Covers 320-1920px viewports, both themes/width preferences, resizing, citations, streaming/finalization/reload, and keyboard scrolling to the last column.
  • python .\functional_tests\test_docs_site_quality.py - 6/6 checks passed.
  • git -c core.whitespace=blank-at-eol,blank-at-eof,space-before-tab,cr-at-eol diff --check - passed.
  • Visually inspected desktop and narrow-window results. The new desktop regression failed on the original code, where the table extended to approximately 4533px while the bubble ended at 1040px.

Known baseline failures: The adjacent orchestration streaming suite has missing pending-input API stubs and an outdated Review the running plan button expectation. python -m pytest .\ui_tests\test_v2_orchestration_streaming_bubble.py -q -k "a_run_shows_only" --tb=short reproduced 1 assertion failure and 1 teardown error with all three production UI files restored to their original source. These unrelated tests were left unchanged. UI detector findings concern an existing activity animation and a false positive matching <img> in an accessibility comment, not this change.

Documentation

  • Release notes updated, or not needed
  • Feature documentation updated, or not needed
  • Fix documentation updated, or not needed

Added docs/explanation/fixes/V2_CHAT_TABLE_OVERFLOW_FIX.md. Release notes remain unchanged: approval was requested, but the user was unavailable. No new capability or control requires feature documentation.

Security checklist

  • New Flask routes include @swagger_route(security=get_auth_security())
  • Settings sent to non-admin frontends use sanitize_settings_for_user()
  • Browser JavaScript is served from local SimpleChat static assets only; no CDN-hosted JS
  • No secrets, keys, connection strings, or local-only artifacts are included

No routes or settings-response boundaries changed. Markdown keeps semantic React rendering with raw HTML disabled. Build outputs, screenshots, and other local test artifacts are not committed.

Expand table-bearing assistant replies within the conversation, wrap long cell content, and contain remaining overflow in keyboard-accessible table regions. Add responsive regression coverage and fix documentation, and bump the app to 0.261.425.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@paullizer
Paul Lizer (paullizer) merged commit f292209 into microsoft:Development Nov 10, 2026
13 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.

1 participant