[remote-dbg-next] Extract renderField method to dedicated component

RESOLVED FIXED in Firefox 65

Status

P1
normal
RESOLVED FIXED
6 months ago
4 months ago

People

(Reporter: jdescottes, Assigned: ladybenko)

Tracking

(Blocks: 1 bug)

unspecified
Firefox 65

Firefox Tracking Flags

(firefox65 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

6 months ago
As discussed while reviewing l10n changes, having a dedicated component instead of the renderField method would enable us to use fluent's Localized component rather than abusing withLocalization::getString.

https://searchfox.org/mozilla-central/rev/0640ea80fbc8d48f8b197cd363e2535c95a15eb3/devtools/client/aboutdebugging-new/src/components/debugtarget/ExtensionDetail.js#21

https://searchfox.org/mozilla-central/rev/0640ea80fbc8d48f8b197cd363e2535c95a15eb3/devtools/client/aboutdebugging-new/src/components/debugtarget/WorkerDetail.js#33

With such a component we could rewrite the localization as

Localized(
  { 
    id: "about-debugging-something-field",
    attrs: { name: true, title: true }
  },
  Field(
    {
      key: "something"
      name: "Fallback name",
      title: "Fallback title"
    },
    content
  )
)
(Reporter)

Updated

5 months ago
Blocks: 1494541
No longer blocks: 1462208
(Assignee)

Updated

5 months ago
Assignee: nobody → balbeza
Status: NEW → ASSIGNED
Priority: P3 → P1

Comment 2

4 months ago
Pushed by balbeza@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9948f43917b0
Extract renderField method to dedicated component. r=jdescottes,daisuke

Comment 3

4 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/9948f43917b0
Status: ASSIGNED → RESOLVED
Last Resolved: 4 months ago
status-firefox65: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 65
You need to log in before you can comment on or make changes to this bug.