Closed Bug 1861688 Opened 2 years ago Closed 1 year ago

Can't change moz-label's or moz-support-link's fluent strings in storybook

Categories

(Toolkit :: UI Widgets, defect, P3)

Desktop
All
defect

Tracking

()

RESOLVED FIXED
143 Branch
Tracking Status
firefox121 --- wontfix
firefox143 --- fixed

People

(Reporter: Gijs, Assigned: cmeade)

References

(Blocks 1 open bug)

Details

(Whiteboard: [recomp] [acorntractors])

Attachments

(1 file)

Hanna mentioned that this storybook may predate the integration of the fluent plugin in storybook, and that may be why you can't change it.

To elaborate/change the scope of this slightly - our stories for moz-label and moz-support-link were both written before we had the fluent Storybook addon and are using fluent in a way that is effectively deprecated. Both of them load fluent files from a /locales-preview directory that we can get rid of by moving those strings into the stories themselves using the fluent parameter we now support (one example here). So the steps are:

  1. change both stories to use the fluent parameter
  2. stop calling insertFTLIfNeeded here and here
  3. remove everything in browser/locales-preview
Blocks: storybook
Summary: Can't change moz-label's fluent string in storybook → Can't change moz-label's or moz-support-link's fluent strings in storybook
Whiteboard: [fidefe-reusable-components]
Severity: -- → S3
Priority: -- → P3
See Also: → 1861687

This could potentially be fixed by supporting fluent files in the Fluent panel (bug 1826123)

See Also: → 1826123
Whiteboard: [fidefe-reusable-components] → [recomp]
Whiteboard: [recomp] → [recomp] [acorntractors]
Assignee: nobody → cmeade
Status: NEW → ASSIGNED
Pushed by tgiles@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/321a517f843c https://hg.mozilla.org/integration/autoland/rev/3e9523df82d6 moz-label and moz-support-link have storybook fluent configuration r=reusable-components-reviewers,mkennedy
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 143 Branch
QA Whiteboard: [qa-triage-done-c144/b143]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: