Closed Bug 1916547 Opened 8 months ago Closed 5 months ago

Render Review Checker in the new sidebar

Categories

(Firefox :: Shopping, task, P3)

task
Points:
5

Tracking

()

RESOLVED FIXED
135 Branch
Tracking Status
firefox135 --- fixed

People

(Reporter: kpatenio, Assigned: fchasen)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fidefe-RCSidebar])

Attachments

(1 file, 2 obsolete files)

Unlike Bug 1916017, this ticket is for actually showing the Review Checker panel in the sidebar, if it is enabled via the sidebar's settings.

Severity: -- → N/A
Whiteboard: [fidefe-RCSidebar]
Assignee: nobody → fchasen
Status: NEW → ASSIGNED
Points: --- → 5
Priority: -- → P3
Attachment #9425948 - Attachment description: WIP: Bug 1916547 - Render Review Checker in the new sidebar. → Bug 1916547 - Render Review Checker in the new sidebar. r?#shopping-reviewers!
Attachment #9425948 - Attachment description: Bug 1916547 - Render Review Checker in the new sidebar. r?#shopping-reviewers! → Bug 1916547 - Part 1: Render Review Checker in the new sidebar. r?#shopping-reviewers!
  • Adds AboutWelcomeReviewChecker actors for displaying AboutWelcome in the sidebar review checker panel (which isn't an aboutpage).
  • AboutWelcomeReviewCheckerParent extends AboutWelcomeParent like the shopping sidebar AboutWelcomeShoppingParent does, but overrides receiveMessage to use embedderElement to get the sidebar browser.
  • AboutWelcomeReviewCheckerChild is just a renamed export of AboutWelcomeShoppingChild.

Note prefs set by the message may not work yet since opening and closing the sidebar is per window now and not controlled by the active pref.

Depends on D222840

Attachment #9426232 - Attachment is obsolete: true
Attachment #9426232 - Attachment is obsolete: false
Attachment #9426232 - Attachment is obsolete: true
Blocks: 1921374
Depends on: 1921471
Attachment #9425948 - Attachment description: Bug 1916547 - Part 1: Render Review Checker in the new sidebar. r?#shopping-reviewers! → WIP: Bug 1916547 - Part 1: Render Review Checker in the new sidebar. r?#shopping-reviewers!
Attachment #9426953 - Attachment description: Bug 1916547 - Part 2: Render AboutWelcome in the new sidebar. r?#shopping-reviewers!,#omc-reviewers! → WIP: Bug 1916547 - Part 2: Render AboutWelcome in the new sidebar. r?#shopping-reviewers!,#omc-reviewers!
Attachment #9425948 - Attachment description: WIP: Bug 1916547 - Part 1: Render Review Checker in the new sidebar. r?#shopping-reviewers! → Bug 1916547 - Part 1: Render Review Checker in the new sidebar. r?#shopping-reviewers!
Attachment #9426953 - Attachment description: WIP: Bug 1916547 - Part 2: Render AboutWelcome in the new sidebar. r?#shopping-reviewers!,#omc-reviewers! → Bug 1916547 - Part 2: Render AboutWelcome in the new sidebar. r?#shopping-reviewers!,#omc-reviewers!
No longer depends on: 1921471
Pushed by fchasen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/020dc75abd7f Part 1: Render Review Checker in the new sidebar. r=shopping-reviewers,firefox-desktop-core-reviewers ,sidebar-reviewers,sclements,mossop,kpatenio
Status: ASSIGNED → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → 135 Branch

Comment on attachment 9426953 [details]
Bug 1916547 - Part 2: Render AboutWelcome in the new sidebar. r?#shopping-reviewers!,#omc-reviewers!

Revision D223431 was moved to bug 1921570. Setting attachment 9426953 [details] to obsolete.

Attachment #9426953 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: