Repository navigation
🪟 feat: Add Welcome Footer and Response Author Options and Tidy the Chat Chrome - #17879
Conversation
|
Lighthouse CI failed. The last 80 log lines contain the measured budgets and assertion failures. |
Codex Review Summary: ✅ CompletedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 96a1d38be1
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2877d456da
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
12aa109 to
50d077e
Compare
50d077e to
f2df943
Compare
Summary
A design review of the chat screen flagged several things that distract from the conversation. Memories opened in a chat follows the user to
/agents, the sidebar's Projects section is collapsed for an account with no projects, its Projects, Pinned and Chats headings sit 12px right of the date labels, and the dot on a streaming response explains nothing to anyone. Each is fixed here.Two more are deployment preferences, so they ship as opt-in
librechat.yamlsettings whose defaults keep today's layout.interface.landingFooter: composerplaces the welcome screen's footer directly under the composer instead of at the bottom of the page.interface.messageAuthor: whenDifferentdrops the avatar and name from a response when the header's model selector already shows that agent. Another agent's response, a subagent thread and a deployment with no selector all keep the author, and screen readers still get it.Type of change
Testing
Each behavior has a tagged scenario in
landing-greeting.spec.tsor the newlaunch-qa.spec.ts. In the running app I confirmed each one in light and dark, with both settings off and on. The settings were served through a mocked/api/config; the path from YAML is covered by the schema,loadDefaultInterfaceand shell bootstrap specs. The finished reply and empty projects list in the screenshots are mocked responses.Tested environments/configuration:
Automated tests:
npx playwright test --config=e2e/playwright.config.mock.tson both specs: passedreviewctl precheckon this branch: passedroutePanels.spec.tsandProjectsSection.spec.tsx; extendedContentRow,EmptyText,chrome,Screen,ChatPane,ChatView,Footer,config-schemas,interfaceandfooterspecsScreenshots / recordings
Before is the base revision; after is this branch, with the settings on where a row names one.
landingFooter: composermessageAuthor: whenDifferent/agentswith Memories openRisk / compatibility
Both settings default to today's behavior. Three changes apply to every deployment: Projects starts expanded, which reverses the empty-section collapse from #13531 for anyone who never toggled it; the Memories rail icon is hidden on
/agents; and registering the button's default size utilities in the app'scnalso gives the mobile drawer's switcher theh-10 px-2it already asks for. WithwhenDifferenton, the hover timestamp and model name go away with the row, though screen readers keep both.Checklist