Skip to content

Commit 828fdd7

Browse files
committed
adding new prop to renderItem accessibilityCollectionItem
- remove the wrapper View and cellstyle flex: 1 - add prop accessibilityCollectionItem to FlatList renderItem to allow users set the accessibilityInformation - update rn-tester flatlist examples Follow up of 1a98f56 Related fabOnReact/react-native-notes#6 (comment)
1 parent 4e64a52 commit 828fdd7

File tree

4 files changed

+22
-17
lines changed

4 files changed

+22
-17
lines changed

Libraries/Lists/FlatList.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -631,9 +631,10 @@ class FlatList<ItemT> extends React.PureComponent<Props<ItemT>, void> {
631631
};
632632

633633
const element = renderer({
634-
item: {...it, accessibilityCollectionItem},
634+
item: it,
635635
index: index * numColumns + kk,
636636
separators: info.separators,
637+
accessibilityCollectionItem,
637638
});
638639

639640
return element != null ? (
@@ -651,7 +652,7 @@ class FlatList<ItemT> extends React.PureComponent<Props<ItemT>, void> {
651652

652653
return renderer({
653654
...info,
654-
item: {...info.item, accessibilityCollectionItem},
655+
accessibilityCollectionItem,
655656
});
656657
}
657658
},
@@ -661,7 +662,7 @@ class FlatList<ItemT> extends React.PureComponent<Props<ItemT>, void> {
661662
_getAccessibilityCollection = () => {
662663
const accessibilityCollectionProps = {
663664
itemCount: this.props.data ? this.props.data.length : 0,
664-
//$FlowFixMe[incompatible-call] see https://bit.ly/3viYSh8
665+
//$FlowFixMe[incompatible-call] see https://bit.ly/3MJiZLL
665666
rowCount: this._getItemCount(this.props.data),
666667
columnCount: numColumnsOrDefault(this.props.numColumns),
667668
hierarchical: false,
@@ -702,7 +703,6 @@ class FlatList<ItemT> extends React.PureComponent<Props<ItemT>, void> {
702703

703704
const styles = StyleSheet.create({
704705
row: {flexDirection: 'row'},
705-
cellStyle: {flex: 1},
706706
});
707707

708708
module.exports = FlatList;

packages/rn-tester/js/examples/FlatList/FlatList-basic.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -280,7 +280,7 @@ class FlatListExample extends React.PureComponent<Props, State> {
280280
return (
281281
<View
282282
importantForAccessibility="yes"
283-
accessibilityCollectionItem={item.accessibilityCollectionItem}
283+
accessibilityCollectionItem={accessibilityCollectionItem}
284284
style={styles.card}>
285285
<ItemComponent
286286
item={item}

packages/rn-tester/js/examples/FlatList/FlatList-multiColumn.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -140,11 +140,11 @@ class MultiColumnExample extends React.PureComponent<
140140
getItemLayout(data, index).length + 2 * (CARD_MARGIN + BORDER_WIDTH);
141141
return {length, offset: length * index, index};
142142
}
143-
_renderItemComponent = ({item}: RenderItemProps<any | Item>) => {
143+
_renderItemComponent = ({item, accessibilityCollectionItem}) => {
144144
return (
145145
<View
146146
importantForAccessibility="yes"
147-
accessibilityCollectionItem={item.accessibilityCollectionItem}
147+
accessibilityCollectionItem={accessibilityCollectionItem}
148148
style={styles.card}>
149149
<ItemComponent
150150
item={item}

packages/rn-tester/js/examples/FlatList/FlatList-nested.js

Lines changed: 15 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -51,20 +51,25 @@ const DATA = [
5151
},
5252
];
5353

54-
const Item = ({ title }) => (
55-
<View style={styles.item}>
56-
<Text style={styles.title}>{title}</Text>
54+
const Item = ({item, accessibilityCollectionItem}) => (
55+
<View
56+
importantForAccessibility="yes"
57+
accessibilityCollectionItem={accessibilityCollectionItem}
58+
style={styles.item}>
59+
<Text style={styles.title}>{item.title}</Text>
5760
</View>
5861
);
5962

60-
const renderItem = ({ item }) => <Item title={item.title} />;
63+
const renderItem = (props) => <Item {...props} />;
6164

62-
const renderFlatList = ({ item }) => (
63-
<View>
64-
<Text>Flatlist {item}</Text>
65-
<FlatList renderItem={renderItem} horizontal data={DATA} />
66-
</View>
67-
);
65+
const renderFlatList = ({item}) => {
66+
return (
67+
<View>
68+
<Text>Flatlist {item}</Text>
69+
<FlatList renderItem={renderItem} horizontal data={DATA} />
70+
</View>
71+
);
72+
};
6873

6974
const FlatListNested = () => {
7075
return (

0 commit comments

Comments
 (0)