Closed Bug 2034921 Opened 2 months ago Closed 26 days ago

Re-enable smart cursor and integrate with Smart Window

Categories

(Core :: Machine Learning: Frontend, task)

task

Tracking

()

VERIFIED FIXED
154 Branch
Tracking Status
relnote-firefox --- 152+
firefox152 --- verified
firefox153 --- verified
firefox154 --- verified

People

(Reporter: jlevinsohn, Assigned: echa)

References

(Blocks 3 open bugs)

Details

(Whiteboard: [aiasst])

Attachments

(3 files)

Re-enable smart cursor and ensure it works within Smart Window using the native sidebar and chat experience.

  • Smart cursor is enabled under pref browser.ml.chat.shortcuts
  • Highlighting text on a webpage surfaces the smart cursor affordance
  • Right-click "Ask AI" entry triggers the same Smart Window flow
  • Behavior matches Classic implementation, adapted to Smart Window UI

Flow 1: Assistant closed

  • Triggering smart cursor opens the Smart Window sidebar
  • Selected text is passed into the assistant
  • User is prompted with actions (e.g. Summarize, Explain this, Quiz me)
  • Selecting an action creates a prompt in the Smart Window chat

Flow 2: Assistant open

  • Triggering smart cursor does not reopen sidebar
  • Selected text is added to the existing Smart Window chat
  • Suggested actions (Summarize, Explain, etc.) are shown inline
  • Selecting an action appends to the current conversation
  • Smart cursor interaction takes priority over natural language request input when triggered
  • Selected text is accurately captured and passed to the assistant
  • Interaction works consistently across:
    • inline shortcut
    • right-click menu

Flow 3: Right click

  • Trigger Ask menu/submenu

  • No regression to existing Smart Window chat behavior

  • No regression to Classic smart cursor behavior outside Smart Window

Figma Specs

Duplicate of this bug: 2003057
QA Contact: rdoghi
Assignee: nobody → echa
Blocks: 2044112
  • Enable Smart Cursor actions in Smart Window behind browser.ml.chat.shortcuts.smartwindow and route selected text to the sidebar chat
  • Update Smart Window context menus/shortcuts to use Smart Window-specific UI and remove unsupported options
Blocks: 2047697
Blocks: 2048401
Pushed by echa@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/6825b3a4dc4e https://hg.mozilla.org/integration/autoland/rev/36629cac1613 Re-enable smart cursor and integrate with Smart Window r=fluent-reviewers,flozia,bolsson
Pushed by asilaghi@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/f3fb67a77402 https://hg.mozilla.org/integration/autoland/rev/6bb372141c07 Revert "Bug 2034921 - Re-enable smart cursor and integrate with Smart Window r=fluent-reviewers,flozia,bolsson" for causing bc failures at browser_genai_actors.js

Backed out for causing bc failures at browser_genai_actors.js
Backout Link
Push with failures
Failure Log
Failure line TEST-UNEXPECTED-FAIL | browser/components/genai/tests/browser/browser_genai_actors.js | test_actor_disabled - Missing expected exception. Actor disabled

Flags: needinfo?(echa)
Pushed by echa@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/948b055e8f6d https://hg.mozilla.org/integration/autoland/rev/2e1d48565c3e Re-enable smart cursor and integrate with Smart Window r=fluent-reviewers,flozia,bolsson
Status: NEW → RESOLVED
Closed: 26 days ago
Resolution: --- → FIXED
Target Milestone: --- → 154 Branch

Verified as fixed in our latest Nightly 154.0a1 (2026-06-19), however it seems that the Ask AI submenus from when we right click inside a page or from when we Right click a Tab are missing from Smart Window, we do have them in Classic Firefox.

Are these being implemented with a different bug ? should I log a seperate one ?

QA Whiteboard: [qa-ver-needed-c154/b153]
Flags: needinfo?(echa)

Hi Rares, as discussed on Slack the right click inside a page (without highlighted text) is not in scope for this iteration. thanks!

Updating the main status flag.

Status: RESOLVED → VERIFIED
QA Whiteboard: [qa-ver-needed-c154/b153] → [qa-ver-done-c154/b153]
Blocks: 2050914
  • Enable Smart Cursor actions in Smart Window behind browser.ml.chat.shortcuts.smartwindow and route selected text to the sidebar chat
  • Update Smart Window context menus/shortcuts to use Smart Window-specific UI and remove unsupported options

Original Revision: https://phabricator.services.mozilla.com/D306022

Attachment #9606462 - Flags: approval-mozilla-beta?

firefox-beta Uplift Approval Request

  • User impact if declined/Reason for urgency: This patch enables Smart Cursor actions (e.g. Summarize, Explain this, Proofread) in Smart Window by routing them to the built-in assistant.
  • Code covered by automated testing?: yes
  • Fix verified in Nightly?: yes
  • Needs manual QE testing?: yes
  • Steps to reproduce for manual QE testing: Open a Smart Window.
    Set prefs browser.ml.chat.shortcuts and browser.ml.chat.shortcuts.smartwindow to true
    Navigate to a page with selectable text.
    Select text and verify the Smart Cursor shortcut appears.
    Choose an action (e.g. Summarize).
    Verify the prompt is submitted to the Smart Window sidebar assistant.
    Verify Classic Window Smart Cursor behavior remains unchanged.
  • Risk associated with taking this patch: low
  • Explanation of risk level: The feature is isolated to Smart Window and is backed by existing Smart Cursor functionality.
  • String changes made/needed?: Yes
  • Is Android affected?: unknown
Flags: qe-verify+
  • Enable Smart Cursor actions in Smart Window behind browser.ml.chat.shortcuts.smartwindow and route selected text to the sidebar chat
  • Update Smart Window context menus/shortcuts to use Smart Window-specific UI and remove unsupported options

Original Revision: https://phabricator.services.mozilla.com/D306022

Attachment #9606463 - Flags: approval-mozilla-release?

firefox-release Uplift Approval Request

  • User impact if declined/Reason for urgency: This patch enables Smart Cursor actions (e.g. Summarize, Explain this, Proofread) in Smart Window by routing them to the built-in assistant.
  • Code covered by automated testing?: yes
  • Fix verified in Nightly?: yes
  • Needs manual QE testing?: yes
  • Steps to reproduce for manual QE testing: Open a Smart Window.
    Set prefs browser.ml.chat.shortcuts and browser.ml.chat.shortcuts.smartwindow to true
    Navigate to a page with selectable text.
    Select text and verify the Smart Cursor shortcut appears.
    Choose an action (e.g. Summarize).
    Verify the prompt is submitted to the Smart Window sidebar assistant.
    Verify Classic Window Smart Cursor behavior remains unchanged.
  • Risk associated with taking this patch: low
  • Explanation of risk level: The feature is isolated to Smart Window and is backed by existing Smart Cursor functionality.
  • String changes made/needed?: Yes
  • Is Android affected?: unknown
QA Whiteboard: [qa-ver-done-c154/b153] → [uplift][qa-ver-needed-c154/b153]
Attachment #9606462 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Flags: in-testsuite+

Verified as fixed in our latest Beta 153.0b12

Attachment #9606463 - Flags: approval-mozilla-release? → approval-mozilla-release+

Release Note Request (optional, but appreciated)
[Why is this notable]: new smart window assistant interaction
[Affects Firefox for Android]: no
[Suggested wording]: Enabled Smart Cursor in Smart Window, allowing users to select text on a page to summarize, explain, or take other actions in the built-in assistant.
Select text on a page and get quick access to summarize, explain, and more using the built-in assistant.
[Links (documentation, blog post, etc)]: https://www.firefox.com/smart-window is needed to activate the feature and it's region restricted, so we should probably show the "progressive roll out" indicator

relnote-firefox: --- → ?

Added to the 152.0.6 relnotes.

Verified as fixed in our latest Release 152.0.6

QA Whiteboard: [uplift][qa-ver-needed-c154/b153] → [uplift][qa-ver-done-c154/b153]
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: