From 27ebe3645420ac7068335216613cd311b569bb49 Mon Sep 17 00:00:00 2001 From: Qstick Date: Wed, 27 Feb 2019 21:53:08 -0500 Subject: [PATCH] Fixed: Select all in TrackFile editor --- .../src/TrackFile/Editor/TrackFileEditorModalContent.js | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/frontend/src/TrackFile/Editor/TrackFileEditorModalContent.js b/frontend/src/TrackFile/Editor/TrackFileEditorModalContent.js index 85576225f..d18a4144b 100644 --- a/frontend/src/TrackFile/Editor/TrackFileEditorModalContent.js +++ b/frontend/src/TrackFile/Editor/TrackFileEditorModalContent.js @@ -1,7 +1,9 @@ import _ from 'lodash'; import PropTypes from 'prop-types'; import React, { Component } from 'react'; +import hasDifferentItems from 'Utilities/Object/hasDifferentItems'; import getSelectedIds from 'Utilities/Table/getSelectedIds'; +import removeOldSelectedState from 'Utilities/Table/removeOldSelectedState'; import selectAll from 'Utilities/Table/selectAll'; import toggleSelected from 'Utilities/Table/toggleSelected'; import { kinds } from 'Helpers/Props'; @@ -60,8 +62,10 @@ class TrackFileEditorModalContent extends Component { } componentDidUpdate(prevProps) { - if (prevProps.items !== this.props.items) { - this.onSelectAllChange({ value: false }); + if (hasDifferentItems(prevProps.items, this.props.items)) { + this.setState((state) => { + return removeOldSelectedState(state, prevProps.items); + }); } }