Format the React and Redux components in RDM

RESOLVED FIXED in Firefox 63

Status

enhancement
P3
normal
RESOLVED FIXED
11 months ago
11 months ago

People

(Reporter: gl, Assigned: gl)

Tracking

unspecified
Firefox 63

Firefox Tracking Flags

(firefox63 fixed)

Details

Attachments

(1 attachment)

We are doing a bit of formatting and reformatting in this patch:

- Sort the propTypes and this.props alphabetically
- Formatting how we return React DOM. 
We typically do:
return dom.div(
  {
     ...
  }
);  
I am introducing braces after the return so that we can correctly indent the React DOM:
return (
  dom.div()
);
- Also another formatting for React DOM. If we only have a single prop for the React component, I am formatting it to be on the same name if possible.
Example: 
dom.span(
  {
    id: "blah"
  }
) 
To: 
dom.span({ id: "blah" })
Attachment #9002888 - Flags: review?(rcaliman)
Attachment #9002888 - Flags: review?(rcaliman) → review+
Pushed by gabriel.luong@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/af1af69c1d9c
Format the React and Redux components in RDM. r=rcaliman
https://hg.mozilla.org/mozilla-central/rev/af1af69c1d9c
Status: ASSIGNED → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
You need to log in before you can comment on or make changes to this bug.