Skip to content

Commit a5af549

Browse files
committed
Pass the itemIndex to the renderItem function
Add itemIndex to test
1 parent a9bc179 commit a5af549

File tree

2 files changed

+4
-5
lines changed

2 files changed

+4
-5
lines changed

src/Item.js

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -51,10 +51,9 @@ export default class Item extends Component {
5151
};
5252

5353
render() {
54-
const { isHighlighted, item, renderItem, renderItemData, ...restProps } = this.props;
54+
const { isHighlighted, item, itemIndex, renderItem, renderItemData, ...restProps } = this.props;
5555

5656
delete restProps.sectionIndex;
57-
delete restProps.itemIndex;
5857

5958
if (typeof restProps.onMouseEnter === 'function') {
6059
restProps.onMouseEnter = this.onMouseEnter;
@@ -74,7 +73,7 @@ export default class Item extends Component {
7473

7574
return (
7675
<li role="option" {...restProps} ref={this.storeItemReference}>
77-
{renderItem(item, { isHighlighted, ...renderItemData })}
76+
{renderItem(item, { itemIndex, isHighlighted, ...renderItemData })}
7877
</li>
7978
);
8079
}

test/plain-list/Autowhatever.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -70,12 +70,12 @@ describe('Plain List Autowhatever', () => {
7070
renderItem.reset();
7171
mouseEnterItem(0);
7272
expect(renderItem).to.have.been.calledOnce;
73-
expect(renderItem).to.be.calledWith({ text: 'Apple' }, { isHighlighted: true });
73+
expect(renderItem).to.be.calledWith({ text: 'Apple' }, { itemIndex: 0, isHighlighted: true });
7474

7575
renderItem.reset();
7676
mouseLeaveItem(0);
7777
expect(renderItem).to.have.been.calledOnce;
78-
expect(renderItem).to.be.calledWith({ text: 'Apple' }, { isHighlighted: false });
78+
expect(renderItem).to.be.calledWith({ text: 'Apple' }, { itemIndex: 0, isHighlighted: false });
7979
});
8080

8181
it('should call `renderItem` once when item is left', () => {

0 commit comments

Comments
 (0)