Skip to content

Add New Chat action and drop chat history overflow#8718

Open
GerardPaligot wants to merge 1 commit into
developfrom
feature/gerard/chat-history-compose-chat
Open

Add New Chat action and drop chat history overflow#8718
GerardPaligot wants to merge 1 commit into
developfrom
feature/gerard/chat-history-compose-chat

Conversation

@GerardPaligot
Copy link
Copy Markdown
Contributor

@GerardPaligot GerardPaligot commented May 29, 2026

Task/Issue URL: https://app.asana.com/1/137249556945/project/72649045549333/task/1215237686779716?focus=true

Description

Reworks the chat history default toolbar per ship review feedback: adds a New Chat action (compose icon, opens Duck.ai for a fresh conversation), drops the overflow entry — whose single "Select" item duplicated the native long-press affordance — and shifts the Fire icon to the trailing edge. The new order is New · Search · Fire. Long-press on a row becomes the canonical entry to select mode.

Steps to test this PR

Note

Prerequisites:

  • Install Internal Debug.
  • Have at least 2 chats in Duck.ai (one pinned) so select mode is reachable.

Happy path — new default toolbar

  • Open the New Tab Page → overflow menu → tap "Chats".
  • Confirm the trailing toolbar icons appear in this order from left to right: New · Search · Fire.
  • Confirm the 3-dot overflow icon is gone from the toolbar.

New Chat action

  • Tap the New icon → confirm Duck.ai opens for a fresh conversation.
  • Return to the chat history screen → confirm the list still renders correctly.

Long-press is the entry to select mode

  • Long-press any chat row → confirm the screen flips to select mode, with that row pre-selected.
  • Confirm the toolbar swaps to back arrow · count · Fire, and that the New and Search icons are hidden.
  • Press additional rows → confirm each toggles into the selection and the count updates.
  • Tap the back arrow → confirm select mode exits and the default toolbar (New · Search · Fire) returns.

UI changes

Before After
image image

Note

Low Risk
UI-only toolbar and navigation wiring; select mode still uses existing long-press and view-model paths.

Overview
Reworks the Duck.ai chat history default toolbar: adds a New action (ic_compose_24) that opens a fresh Duck.ai session via the same path as the empty-state CTA, removes the overflow menu and its Select popup (that flow duplicated long-press), and reorders actions to New · Search · Fire with Fire on the trailing edge.

ChatHistoryFragment wires the new menu item, toggles New/Search visibility in select mode, and deletes showToolbarOverflowPopup plus popup_chat_history_overflow.xml. Copy drops duck_ai_chat_history_action_select and adds duck_ai_chat_history_action_new.

Reviewed by Cursor Bugbot for commit b4b84ae. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown
Contributor Author

GerardPaligot commented May 29, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

Ship review feedback flagged the chat history toolbar's overflow as
redundant — its only entry duplicated the long-press affordance Android
users already discover. The redesign adds a New Chat action so users can
start a fresh conversation directly from this surface, drops the
overflow icon, and shifts Fire to the trailing edge. Long-press becomes
the canonical entry to select mode (already supported by the ViewModel,
so no new wiring is needed).
@GerardPaligot GerardPaligot force-pushed the feature/gerard/chat-history-compose-chat branch from 15a734f to b4b84ae Compare May 29, 2026 13:06
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit b4b84ae. Configure here.

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