Skip to content

docs(thermidor): add button to go back to conversation from search results in UX-aligned demo - #8068

Merged
fbeaudoincoveo merged 10 commits into
mainfrom
KIT-5923-bidirectional-navigation-flow
Jul 27, 2026
Merged

docs(thermidor): add button to go back to conversation from search results in UX-aligned demo#8068
fbeaudoincoveo merged 10 commits into
mainfrom
KIT-5923-bidirectional-navigation-flow

Conversation

@fbeaudoincoveo

@fbeaudoincoveo fbeaudoincoveo commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

https://coveord.atlassian.net/browse/KIT-5923

While this is not in the UX mockup, it will be quite convenient.

Might go in the end, but good addition for now.

back-to-conversation

This PR also fixes a few small bugs I found while testing.

@changeset-bot

changeset-bot Bot commented Jul 24, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: c1d831b

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@fbeaudoincoveo fbeaudoincoveo changed the title docs(thermidor): add button to go back to conversation from search results docs(thermidor): add button to go back to conversation from search results in UX-aligned demo Jul 24, 2026
@svcsnykcoveo

svcsnykcoveo commented Jul 24, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

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

Adds bidirectional navigation UX improvements to the Thermidor demo React app by introducing a “Back to conversation” affordance on the search results page and restructuring view rendering to better preserve state across transitions.

Changes:

  • Add a “Back to conversation” button to the search results header and wire it through AppShell.
  • Rework AppShell rendering to keep the search view mounted (hidden/shown via panels) to preserve search state while navigating.
  • Sync PromptInput internal value when initialValue changes, and align header layout sizing/styles across views.

Reviewed changes

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

Show a summary per file
File Description
samples/thermidor/demo-react/src/index.css Introduces shared layout variable (--header-height) and new view panel classes for preserving mounted views.
samples/thermidor/demo-react/src/components/SearchResultsPage/SearchResultsPage.tsx Adds onBackToConversation prop + renders the new back button.
samples/thermidor/demo-react/src/components/SearchResultsPage/SearchResultsPage.test.tsx Adds unit coverage for the back button behavior.
samples/thermidor/demo-react/src/components/SearchResultsPage/SearchResultsPage.module.css Updates header layout to accommodate the back button and shared header sizing.
samples/thermidor/demo-react/src/components/PromptInput/PromptInput.tsx Syncs internal textarea state when initialValue changes (important for view transitions).
samples/thermidor/demo-react/src/components/ConversationPage/ConversationPage.module.css Aligns conversation header height with the new shared header sizing.
samples/thermidor/demo-react/src/components/AppShell.tsx Wires navigation + persists search view/interface across transitions; adds handler for back-to-conversation.
samples/thermidor/demo-react/src/components/AppShell.bidirectional.test.tsx Adds an end-to-end style unit test suite for bidirectional navigation and persistence behaviors.

@pkg-pr-new

pkg-pr-new Bot commented Jul 24, 2026

Copy link
Copy Markdown
@coveo/atomic

npm i https://pkg.pr.new/@coveo/atomic@8068

@coveo/atomic-hosted-page

npm i https://pkg.pr.new/@coveo/atomic-hosted-page@8068

@coveo/atomic-legacy

npm i https://pkg.pr.new/@coveo/atomic-legacy@8068

@coveo/atomic-react

npm i https://pkg.pr.new/@coveo/atomic-react@8068

@coveo/auth

npm i https://pkg.pr.new/@coveo/auth@8068

@coveo/bueno

npm i https://pkg.pr.new/@coveo/bueno@8068

@coveo/create-atomic

npm i https://pkg.pr.new/@coveo/create-atomic@8068

@coveo/create-atomic-component

npm i https://pkg.pr.new/@coveo/create-atomic-component@8068

@coveo/create-atomic-component-project

npm i https://pkg.pr.new/@coveo/create-atomic-component-project@8068

@coveo/create-atomic-result-component

npm i https://pkg.pr.new/@coveo/create-atomic-result-component@8068

@coveo/create-atomic-rollup-plugin

npm i https://pkg.pr.new/@coveo/create-atomic-rollup-plugin@8068

@coveo/create-ui

npm i https://pkg.pr.new/@coveo/create-ui@8068

@coveo/headless

npm i https://pkg.pr.new/@coveo/headless@8068

@coveo/headless-react

npm i https://pkg.pr.new/@coveo/headless-react@8068

@coveo/relay

npm i https://pkg.pr.new/@coveo/relay@8068

@coveo/shopify

npm i https://pkg.pr.new/@coveo/shopify@8068

commit: c1d831b

@chromatic-com

chromatic-com Bot commented Jul 24, 2026

Copy link
Copy Markdown

Important

UI Tests need review – Review now

🟡 UI Tests: 1 visual and accessibility change must be accepted as baseline
🟢 UI Review: 465 stories published -- no changes
Storybook icon Storybook Publish: 465 stories published

@github-actions

Copy link
Copy Markdown
Contributor

🔗 Scratch Orgs ready to test this PR:

louis-bompart
louis-bompart previously approved these changes Jul 27, 2026

@louis-bompart louis-bompart left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm, css could benefits from some 🧹 eventually

Base automatically changed from KIT-5922-conversation-mode to main July 27, 2026 14:01
@fbeaudoincoveo
fbeaudoincoveo dismissed louis-bompart’s stale review July 27, 2026 14:01

The base branch was changed.

@lavoiesl lavoiesl left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

All demo code, not production code. Approving to unblock progress.

@fbeaudoincoveo
fbeaudoincoveo added this pull request to the merge queue Jul 27, 2026
Merged via the queue into main with commit 488490e Jul 27, 2026
47 checks passed
@fbeaudoincoveo
fbeaudoincoveo deleted the KIT-5923-bidirectional-navigation-flow branch July 27, 2026 18:32
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