Skip to content

Commit e554a1d

Browse files
futuremintjacobroschen
authored andcommitted
Take better advantage of lodash.findIndex
Co-Authored-By: jacobroschen <[email protected]>
1 parent 3e9a7fc commit e554a1d

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

components/tree/private/item.jsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,13 +37,13 @@ const handleSelect = ({ event, props, fromFocus }) => {
3737

3838
const findNextNode = (flattenedNodes, node) => {
3939
const nodes = flattenedNodes.map((flattenedNode) => flattenedNode.node);
40-
const index = findIndex(nodes, (item) => item.id === node.id);
40+
const index = findIndex(nodes, { id: node.id });
4141
return flattenedNodes[(index + 1) % flattenedNodes.length];
4242
};
4343

4444
const findPreviousNode = (flattenedNodes, node) => {
4545
const nodes = flattenedNodes.map((flattenedNode) => flattenedNode.node);
46-
let index = findIndex(nodes, (item) => item.id === node.id) - 1;
46+
let index = findIndex(nodes, { id: node.id }) - 1;
4747
if (index < 0) {
4848
index += flattenedNodes.length;
4949
}
@@ -84,7 +84,7 @@ const handleKeyDownUp = (event, props) => {
8484

8585
const handleKeyDownLeft = (event, props) => {
8686
const nodes = props.flattenedNodes.map((flattenedNode) => flattenedNode.node);
87-
const index = findIndex(nodes, (item) => item.id === props.parent.id);
87+
const index = findIndex(nodes, { id: props.parent.id });
8888
if (index !== -1) {
8989
props.onExpand({
9090
event,

components/tree/private/render-branch.jsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -111,13 +111,13 @@ const handleSelect = ({ event, props, fromFocus }) => {
111111

112112
const findNextNode = (flattenedNodes, node) => {
113113
const nodes = flattenedNodes.map((flattenedNode) => flattenedNode.node);
114-
const index = findIndex(nodes, (item) => item.id === node.id);
114+
const index = findIndex(nodes, { id: node.id });
115115
return flattenedNodes[(index + 1) % flattenedNodes.length];
116116
};
117117

118118
const findPreviousNode = (flattenedNodes, node) => {
119119
const nodes = flattenedNodes.map((flattenedNode) => flattenedNode.node);
120-
let index = findIndex(nodes, (item) => item.id === node.id) - 1;
120+
let index = findIndex(nodes, { id: node.id }) - 1;
121121
if (index < 0) {
122122
index += flattenedNodes.length;
123123
}
@@ -173,7 +173,7 @@ const handleKeyDownLeft = (event, props) => {
173173
const nodes = props.flattenedNodes.map(
174174
(flattenedNode) => flattenedNode.node
175175
);
176-
const index = findIndex(nodes, (item) => item.id === props.parent.id);
176+
const index = findIndex(nodes, { id: props.parent.id });
177177
if (index !== -1) {
178178
props.onExpand({
179179
event,

0 commit comments

Comments
 (0)