Closed Bug 2066474 Opened 1 month ago Closed 23 days ago

Lists widget switcher menu no longer shows a checkmark on the selected list

Categories

(Firefox :: New Tab Page, defect)

defect

Tracking

()

RESOLVED FIXED
158 Branch
Tracking Status
firefox158 --- fixed

People

(Reporter: ini, Assigned: ini, NeedInfo)

Details

Attachments

(2 files)

STR

  1. Set browser.newtabpage.activity-stream.widgets.system.lists.enabled to true
  2. From the widget context menu, create a new list
  3. Click on the drop-down arrow next to the list title
    Expected: there should be a checkmark next to the currently selected list

Issue: no checkmark is visible

The switcher passes checked before type="checkbox", and React assigns custom element props in source order, so panel-item's checked setter runs while type is still unset and no-ops — the checked attribute is never written, so nothing matches :host([type="checkbox"][checked]). Every other checkbox panel-item in newtab orders type first, which is why the Stocks list dropdown still checks.

Summary: Lists widget switcher menu no longer shows a chekmark on the selected list → Lists widget switcher menu no longer shows a checkmark on the selected list
Assignee: nobody → ini
Status: NEW → ASSIGNED

The severity field is not set for this bug.
:thecount, could you have a look please?

For more information, please visit BugBot documentation.

Flags: needinfo?(sdowne)
Pushed by asilaghi@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/3998bffd3c55 https://hg.mozilla.org/integration/autoland/rev/d8e37e76dddb Revert "Bug 2066474 - Restore selected checkmark on lists widget switcher menu. r=maxx" for causing node failures

Backed out for causing node failures
Backout Link
Push with failures
Failure Log
Failure line TEST-UNEXPECTED-FAIL | karma /builds/worker/checkouts/gecko/browser/extensions/newtab | activity-stream:<Lists> marks only the selected list as checked in the switcher: expected undefined to equal false

Flags: needinfo?(ini)
Status: ASSIGNED → RESOLVED
Closed: 23 days ago
Resolution: --- → FIXED
Target Milestone: --- → 158 Branch
Flags: needinfo?(ini)
QA Whiteboard: [qa-triage-done-c159/b158]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: