Closed Bug 1850826 Opened 1 year ago Closed 1 year ago

Refactor LegalParagraph as LinkParagraph & replace usage of cta_paragraph in shopping sidebar onboarding

Categories

(Firefox :: Messaging System, enhancement, P1)

enhancement

Tracking

()

VERIFIED FIXED
119 Branch
Iteration:
119.2 - Sept 11 - Sept 22
Tracking Status
firefox119 --- verified

People

(Reporter: emcminn, Assigned: nsauermann)

References

(Blocks 1 open bug)

Details

(Whiteboard: [omc])

Attachments

(1 file)

As discussed in https://phabricator.services.mozilla.com/D186579; we should repurpose the LegalParagraph component to cover the use of cta_paragraph in the shopping onboarding message - this can be worked on once https://phabricator.services.mozilla.com/D186499 lands.

Whiteboard: [omc]
Priority: -- → P2
See Also: → 1850924

With Bug 1847641 fix in Nightly, we can remove cta_paragraph and use array of text to display second paragraph text something like below in JSON:

above_button_content: [ { type: "text", text: { string_id: "shopping-onboarding-body", }, link_keys: ["learn_more_link"], }, { type: "image", url: "chrome://browser/content/shopping/assets/temp-fakespot-rating.svg", height: "auto", }, { type: "text", text: { string_id: "shopping-onboarding-opt-in-privacy-policy-and-terms-of-use", }, link_keys: ["privacy_policy", "terms_of_use"], }, ], learn_more_link: { action: { type: "OPEN_URL", data: { args: "https://www.support.mozilla.org", where: "tab", }, }, }, privacy_policy: { action: { type: "OPEN_URL", data: { args: "https://www.fakespot.com/privacy-policy", where: "tab", }, }, }, terms_of_use: { action: { type: "OPEN_URL", data: { args: "https://www.fakespot.com/terms", where: "tab", }, }, },

There will be small code change to make LegalParagraph pick style from JSON config that is show text as 'regular' or 'legal' with regular text styled with default
font-size: 0.8em; color: #5B5B66;

and .welcome-text margins adjusted as per shopping figma. This should resolve Bug 1850924

Blocks: 1850924
See Also: 1850924
Assignee: nobody → nsauermann
Status: NEW → ASSIGNED
Iteration: --- → 119.2 - Sept 11 - Sept 22
Pushed by nsauermann@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/695253c15da8 Refactor LegalParagraph as LinkParagraph & replace usage of cta_paragraph in shopping sidebar onboarding r=omc-reviewers,pdahiya,fluent-reviewers,flod,shopping-reviewers,Gijs
Regressions: 1853979
Priority: P2 → P1
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 119 Branch
Duplicate of this bug: 1850924

Verified fixed using Firefox Nightly 119.0a1 (20230925093027) on Windows 10, MacOS 11 and Ubuntu 20.04.

Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: