docs(thermidor): add button to go back to conversation from search results in UX-aligned demo - #8068
Conversation
|
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
There was a problem hiding this comment.
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
AppShellrendering to keep the search view mounted (hidden/shown via panels) to preserve search state while navigating. - Sync
PromptInputinternal value wheninitialValuechanges, 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. |
@coveo/atomic
@coveo/atomic-hosted-page
@coveo/atomic-legacy
@coveo/atomic-react
@coveo/auth
@coveo/bueno
@coveo/create-atomic
@coveo/create-atomic-component
@coveo/create-atomic-component-project
@coveo/create-atomic-result-component
@coveo/create-atomic-rollup-plugin
@coveo/create-ui
@coveo/headless
@coveo/headless-react
@coveo/relay
@coveo/shopify
commit: |
|
Important UI Tests need review – Review now🟡 UI Tests: 1 visual and accessibility change must be accepted as baseline |
louis-bompart
left a comment
There was a problem hiding this comment.
lgtm, css could benefits from some 🧹 eventually
The base branch was changed.
…state across transitions
4667f13 to
c1d831b
Compare
lavoiesl
left a comment
There was a problem hiding this comment.
All demo code, not production code. Approving to unblock progress.
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.
This PR also fixes a few small bugs I found while testing.