Merge pull request #149 from urbit/fix/warnings
fix react key warnings
commitf0e7b227969be48a491786d9809f1c5a77a044cc
authorMatt Condon <mattgcondon@gmail.com>
dateMon 10 Jun 2019, 15:18 (7 years ago)
parent181f4f7aadee95
src/bridge/components/InnerLabelDropdown.js+3−3
@@ -33,12 +33,12 @@ class InnerLabelDropdown extends React.Component {
3333 }
3434
3535 getOptionElems() {
36− return this.props.options.map(opt => {
36+ return this.props.options.map((opt, i) => {
3737 if (opt.type === "divider") {
38− return <DropdownDivider />
38+ return <DropdownDivider key={`divider-${i}`} />
3939 } else {
4040 return (
41− <DropdownItem onClick={() => this.selectOption(opt.value)}>
41+ <DropdownItem key={opt.title} onClick={() => this.selectOption(opt.value)}>
4242 { opt.title }
4343 </DropdownItem>
4444 )
src/bridge/components/PointList.js+1
@@ -15,6 +15,7 @@ const PointList = props => {
1515 {
1616 lodash.map(chunk, point =>
1717 <RenderedPoint
18+ key={ point }
1819 setPointCursor={ setPointCursor }
1920 routeHandler={ routeHandler }
2021 point={ point }
src/walletgen/components/Header.js+1−1
@@ -40,7 +40,7 @@ const ProgressIndicator = ({ currentStep, totalSteps }) =>
4040 <div className={'absolute ml-3 h-9 flex align-center text-sm white text-600'}>{`Step ${currentStep} of ${totalSteps}`}</div>
4141 <div className={'flex'}>
4242 {
43− seq(totalSteps).map((step, index) => <div className={`${ index+1 > currentStep ? 'bg-gray' : 'bg-blue' } h-9 col-`}>{''}</div> )
43+ seq(totalSteps).map((step, index) => <div key={index} className={`${ index+1 > currentStep ? 'bg-gray' : 'bg-blue' } h-9 col-`}>{''}</div> )
4444 }
4545 </div>
4646