Timeline: Add role='row' to each element item rendered

This commit is contained in:
Scott Nonnenberg 2019-09-16 12:07:57 -07:00
parent 0e25b405e1
commit 1984ffc94c

View file

@ -510,13 +510,13 @@ export class Timeline extends React.PureComponent<Props, State> {
if (!haveOldest && row === 0) { if (!haveOldest && row === 0) {
rowContents = ( rowContents = (
<div data-row={row} style={styleWithWidth}> <div data-row={row} style={styleWithWidth} role="row">
{renderLoadingRow(id)} {renderLoadingRow(id)}
</div> </div>
); );
} else if (oldestUnreadRow === row) { } else if (oldestUnreadRow === row) {
rowContents = ( rowContents = (
<div data-row={row} style={styleWithWidth}> <div data-row={row} style={styleWithWidth} role="row">
{renderLastSeenIndicator(id)} {renderLastSeenIndicator(id)}
</div> </div>
); );
@ -526,6 +526,7 @@ export class Timeline extends React.PureComponent<Props, State> {
data-row={row} data-row={row}
className="module-timeline__message-container" className="module-timeline__message-container"
style={styleWithWidth} style={styleWithWidth}
role="row"
> >
{renderTypingBubble(id)} {renderTypingBubble(id)}
</div> </div>
@ -544,6 +545,7 @@ export class Timeline extends React.PureComponent<Props, State> {
data-row={row} data-row={row}
className="module-timeline__message-container" className="module-timeline__message-container"
style={styleWithWidth} style={styleWithWidth}
role="row"
> >
{renderItem(messageId, this.props)} {renderItem(messageId, this.props)}
</div> </div>