Skip to content

Conversation

LukasPaczos
Copy link
Contributor

@LukasPaczos LukasPaczos commented Aug 14, 2025

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

Description

This PR updates the Duck.ai / AI Features page.

There are 3 primary commits, and I recommend reviewing the PR commit-by-commit which might be easier:

  1. d388dae - updates the settings activity title, activity main logo/icon, and update to the main Duck.ai toggle's copy.
  2. ee2ad82 - moves Duck.ai shortcut toggles to a dedicated screen.
  3. 7acbaa4 - updates the Input Screen toggle to a pair of radio buttons with pictograms.

The above commits also update string resources in some way, so I'll open a separate translations PR once copy is approved that I'll merge into this PR before merging it all to develop. Adding a "do not merge" label until that's done, but otherwise the PR is ready for review.

Steps to test this PR

  • Install a clean, internal build.
  • Verify the Input Screen is disabled (no Search/Duck.ai switch when focusing on the omnibar).
  • Verify there's a Duck.ai button directly in the address bar.
  • Open "Settings -> AI Feature".
  • Verify that activity title is AI Features.
  • Verify that the logo is the new "AI Features" logo.
  • Verify the primary Duck.ai toggle copy matches the design.
  • Change the setting to "Search & Duck.ai (Experimental)".
  • Click "Share Feedback" and verify that the feedback form opens.
  • Go back and click "Duck.ai Shortcuts".
  • Verify that a new activity with Menu and Address Bar toggles opens.
  • Disable Address Bar toggle.
  • Go back to the browser.
  • Verify there's no Duck.ai button in the address bar.
  • Verify that focusing on the omnibar opens the Input Screen with Search.Duck.ai switch.

UI changes

Before After
before-changes rebranding_light

Copy link
Collaborator

@aitorvs aitorvs left a comment

Choose a reason for hiding this comment

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

LGTM, left one nit comment.

Also tested and works as expected

@LukasPaczos
Copy link
Contributor Author

Translation job is running in #6606.

@LukasPaczos LukasPaczos force-pushed the featre/lukasz-p/update-ai-settings-page branch from a48ea12 to 4fad082 Compare August 19, 2025 12:48
@LukasPaczos LukasPaczos force-pushed the featre/lukasz-p/update-ai-settings-page branch from 8ecfd11 to 4d96b04 Compare August 19, 2025 14:40
@LukasPaczos
Copy link
Contributor Author

Still waiting for the final translations to land, but since the initial approval I've made some minor changes based on the Ship Review feedback - @aitorvs would you be able to give the new commits a quick pass?

  • 9341cac - updates copy
  • de79e0c + 4fad082 - updates icons used
  • 7d045b2 - aligns the content to the end of the main toggle's icon, if rebranding enabled
  • 4d96b04 - updates text type for sub-titles

Translations for #6586.

---------

Co-authored-by: Dax The Translator <[email protected]>
@LukasPaczos LukasPaczos merged commit 680346a into develop Aug 21, 2025
8 checks passed
@LukasPaczos LukasPaczos deleted the featre/lukasz-p/update-ai-settings-page branch August 21, 2025 14:39
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.

2 participants