r/p5.js-web-editor
1
0
Fork 0
Code Issues 1 Pull requests Releases 1 Wiki Activity

add key property to lint message elements

Browse source
This commit is contained in:
omnikrll 2016-08-23 23:38:58 -04:00
parent de62acdaaa
commit 5d11dcdc84
1 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
client/modules/IDE/components/EditorAccessibility.js
View file

@ -9,7 +9,7 @@ class EditorAccessibility extends React.Component {
if (this.props.lintMessages.length > 0) { if (this.props.lintMessages.length > 0) {
for (let i = 0; i < this.props.lintMessages.length; i++) { for (let i = 0; i < this.props.lintMessages.length; i++) {
messages.push( messages.push(
<li> <li key={i}>
{this.props.lintMessages[i].severity} in line {this.props.lintMessages[i].severity} in line
{this.props.lintMessages[i].line} : {this.props.lintMessages[i].line} :
{this.props.lintMessages[i].message} {this.props.lintMessages[i].message}
@ -18,7 +18,12 @@ class EditorAccessibility extends React.Component {
} }
} else { } else {
messages.push( messages.push(
<p tabIndex="0"> There are no lint messages </p> // react wants dom items from an array or
// iterator to have a key property. since this
// is the only item we're pushing to the array
// and don't have a counter to include,
// let's just call it 0.
<p tabIndex="0" key={0}> There are no lint messages </p>
); );
} }
return ( return (