1497099 - [remote-dbg-next] Use common PropTypes in React components

Status: RESOLVED FIXED

(DevTools :: about:debugging, enhancement, P1)

Description

[Daisuke Akatsuka (:daisuke)]

Currently, we define PropTypes in React components respectively. Therefore, if the kind of datas that are sent from reducer has changed, all components that refer need to reflect. Also, a concern of consistency with reducer remains.
To resolve this, manage PropTypes in one place, the each components refer there.
e.g. https://searchfox.org/mozilla-central/source/devtools/client/inspector/fonts/types.js

Comment 1

[Daisuke Akatsuka (:daisuke)]

Attachment: Bug 1497099: Define and use ui state related prop types. r=jdescottes

Comment 2

[Daisuke Akatsuka (:daisuke)]

Attachment: Bug 1497099: Define and use debug target related prop types. r=jdescottes

Depends on D8693

Comment 3

[Daisuke Akatsuka (:daisuke)]

Attachment: Bug 1497099: Define and use runtime related prop types. r=jdescottes

Depends on D8694

Comment 4

[Daisuke Akatsuka (:daisuke)]

As we discussed on slack, we will go forward step by step. Firstly, we define `runtime` object. Then if needed, address next one.

Comment 5

[Daisuke Akatsuka (:daisuke)]

Attachment: Bug 1497099: Use common runtime PropTypes. r=jdescottes

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/32cff4bae820
Use common runtime PropTypes. r=jdescottes

Comment 7

[Eliza Balazs [:ebalazs_]]

https://hg.mozilla.org/mozilla-central/rev/32cff4bae820