2048338 - Add a single row of Shortcuts on Smart Window New Tab

Status: VERIFIED FIXED

(Core :: Machine Learning: Frontend, task)

Description

[Punam Dahiya [:pdahiya]]

Scope of this task is to add a single row of Shortcuts on Smart Window New Tab as per requirement doc here. Shortcut tiles open the destination when clicked.

Figma

Comment 1

[Punam Dahiya [:pdahiya]]

Attachment: Bug 2048338 - Smart Window Full Page Single Row Topsites

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/c052f6e94196
https://hg.mozilla.org/integration/autoland/rev/5d68d101d41b
Smart Window Full Page Single Row Topsites r=gcardieri

Comment 3

[amarc]

https://hg.mozilla.org/mozilla-central/rev/5d68d101d41b

Comment 4

[Punam Dahiya [:pdahiya]]

[Tracking Requested - why for this release]:
Ship the diary feedback shortcuts in the Fx152 dot release to quickly validate whether the lack of shortcuts is creating user friction and contributing to retention issues.

Comment 5

[Punam Dahiya [:pdahiya]]

NI @rdoghi to help verify fix in Nightly before requesting beta and release uplifts thanks!

Comment 6

[Rares Doghi, Desktop QA]

This issue is verified as fixed in our latest Nightly build 154.0a1 (2026-06-28), however we are missing the context menus for them, not to mention there should be a separate option in Settings for them where the user can remove them from the page.
Theres a case where broken urls will make their way on that new tab page and there is no way for a user to get rid of them. Is this something that will be implemented with another bug ? Should the preferences from Settings affect Smart Window as well ?

Comment 7

[Punam Dahiya [:pdahiya]]

(In reply to Rares Doghi, Desktop QA from comment #6)

This issue is verified as fixed in our latest Nightly build 154.0a1 (2026-06-28), however we are missing the context menus for them, not to mention there should be a separate option in Settings for them where the user can remove them from the page.
Theres a case where broken urls will make their way on that new tab page and there is no way for a user to get rid of them. Is this something that will be implemented with another bug ? Should the preferences from Settings affect Smart Window as well ?

Thanks @rdoghi , this first iteration of bringing shortcuts in SW is minimal with no context menu. There is bug 2048434 created for Settings option but triaged not a blocker for initial launch. For now User can hide TopSites by flipping pref browser.smartwindow.hideTopSites

For broken Urls edge cases user can still remove (though not ideal) those from CW New Tab. I will NI product for the next steps at least the initial version TopSites settings stays separate for SW and CW

Comment 8

[Pascal Chevrel (relman team) -> :pascalc]

Punam, are you planning to request uplift? Thanks

Comment 9

[Punam Dahiya [:pdahiya]]

Hi @pascalc yes will submit uplift request today aiming beta and 152 July 14 dot release thanks!

Comment 10

[Punam Dahiya [:pdahiya]]

Attachment: Bug 2048338 - Smart Window Full Page Single Row Topsites

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

Comment 11

[Phabricator Automation]

firefox-beta Uplift Approval Request

• User impact if declined/Reason for urgency: Fix implements the diary feedback shortcuts in the Fx152 dot release to quickly validate whether the lack of shortcuts is creating user friction and contributing to retention issues.
• Code covered by automated testing?: yes
• Fix verified in Nightly?: yes
• Needs manual QE testing?: yes
• Steps to reproduce for manual QE testing: See https://phabricator.services.mozilla.com/D307407 Test Plan
• Risk associated with taking this patch: low
• Explanation of risk level: Fix aiming Smart Window feature top sites display on newtab
• String changes made/needed?: None
• Is Android affected?: no

Comment 12

[Punam Dahiya [:pdahiya]]

Attachment: Bug 2048338 - Smart Window Full Page Single Row Topsites

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

Comment 13

[Phabricator Automation]

firefox-release Uplift Approval Request

• User impact if declined/Reason for urgency: Diary feedback shortcuts in the Fx152 dot release to quickly validate whether the lack of shortcuts is creating user friction and contributing to retention issues.
• Code covered by automated testing?: yes
• Fix verified in Nightly?: yes
• Needs manual QE testing?: yes
• Steps to reproduce for manual QE testing: See https://phabricator.services.mozilla.com/D307407 Test Plan
• Risk associated with taking this patch: low
• Explanation of risk level: Fix aiming Smart Window feature top sites display on newtab
• String changes made/needed?: None
• Is Android affected?: no

Comment 14

[Pulsebot]

https://github.com/mozilla-firefox/firefox/commit/220c6a6c664b
https://hg.mozilla.org/releases/mozilla-beta/rev/27811f3a9954

Comment 15

[Rares Doghi, Desktop QA]

Verified as fixed in our latest Beta 153.0b10

Comment 16

[Pulsebot]

https://github.com/mozilla-firefox/firefox/commit/5a6dade80813
https://hg.mozilla.org/releases/mozilla-release/rev/c7a50b5182cf

Comment 17

[Ed Lee :Mardak]

Release Note Request (optional, but appreciated)
[Why is this notable]: new smart window feature closer to classic parity
[Affects Firefox for Android]: no
[Suggested wording]: The Smart Window assistant now shows a row of shortcuts for quick access to your sites.
Added a row of shortcuts to New Tab, making it easier to get back to sites.
[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

Comment 18

[Ryan VanderMeulen [:RyanVM]]

Added to the 152.0.6 relnotes.

Comment 19

[Rares Doghi, Desktop QA]

Verified as fixed in our latest Release 152.0.6