2035622 - Always show an add shortcut button for New Tab shortcuts

Status: REOPENED

(Firefox :: New Tab Page, task)

Comment 1

[Irene Ni [:ini]]

Attachment: Bug 2035622 - Always show an add shortcut button for New Tab shortcuts. r=thecount

Comment 2

[Irene Ni [:ini]]

Attachment: Bug 2035622 - Make add shortcut button part of shortcuts arrow key navigation order. r=#home-newtab-reviewers

Comment 3

[Irene Ni [:ini]]

Attachment: Bug 2035622 - Include option to Add in New Tab shortcuts context menu. r=#home-newtab-reviewers

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/4eb5d723d627
https://hg.mozilla.org/integration/autoland/rev/c9b51eea20db
Always show an add shortcut button for New Tab shortcuts. r=thecount

Comment 5

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/c9b51eea20db

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/5748b16d3f0f
https://hg.mozilla.org/integration/autoland/rev/6e85ffe7b0fe
Revert "Bug 2035622 - Always show an add shortcut button for New Tab shortcuts. r=thecount" for causing M-trainhop mochitests failures.

Comment 7

[Serban Stanca [:SerbanS]]

Reverted this because it was causing M-trainhop mochitests failures.

• Revert link
• Push with failures
• Failure Log
• Failure line: TEST-UNEXPECTED-FAIL | browser/components/asrouter/tests/browser/browser_asrouter_newtab_messages.js | test_show_newtab_message - Uncaught exception in test - Waiting for asrouter-newtab-message - timed out after 50 tries.

Comment 8

[amarc]

backout merged to central: https://hg.mozilla.org/mozilla-central/rev/6e85ffe7b0fe