1418250 - JSON Viewer to use ES6 Classes, prop-types and react-dom-factories

Status: RESOLVED FIXED

(DevTools :: JSON Viewer, enhancement, P2)

Description

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

There are a number of reasons for this:

1. We need to have the modules available for code from devtools-core, which uses them.
2. It begins the process of getting rid of React deprecation warnings.
3. It prepares us to upgrade to React 16.

Versioning info from devtools-core github repo:

"prop-types": "^15.6.0"
"react-dom-factories": "^1.0.2"

Comment 1

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

Attachment: Bug 1418250 - JSON Viewer to ES6 Classes, prop-types and react-dom-factories

Review commit: https://reviewboard.mozilla.org/r/200724/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/200724/

Comment 2

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Comment on attachment 8929393 [details]
Bug 1418250 - JSON Viewer to ES6 Classes, prop-types and react-dom-factories

https://reviewboard.mozilla.org/r/200724/#review205934

Looks good,

R+ assuming my comments are resolved and try is green.

Thanks Mike!
Honza

::: devtools/client/jsonview/components/HeadersPanel.js:88
(Diff revision 1)
>          )
>        );
> -    },
> -  }));
> +    }
> +  }
> +
> +  let HeadersToolbarFactory = createFactory(HeadersToolbar);

I would prefer creating the factory at at the top of the file.

::: devtools/client/jsonview/components/JsonPanel.js:185
(Diff revision 1)
>          )
>        );
> -    },
> -  }));
> +    }
> +  }
> +
> +  let JsonToolbarFactory = createFactory(JsonToolbar);

Again, please move to the top of the file

::: devtools/client/jsonview/components/TextPanel.js:110
(Diff revision 1)
>          )
>        );
> -    },
> -  }));
> +    }
> +  }
> +
> +  let TextToolbarFactory = createFactory(TextToolbar);

Just like in previous cases.

Comment 3

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

Comment on attachment 8929393 [details]
Bug 1418250 - JSON Viewer to ES6 Classes, prop-types and react-dom-factories

https://reviewboard.mozilla.org/r/200724/#review205934

I have moved the requests to split the components into their own files to bug 1418320

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9fa79b8134eb
JSON Viewer to ES6 Classes, prop-types and react-dom-factories r=Honza

Comment 5

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/9fa79b8134eb