Merge pull request #280 from creative-commoners/pulls/2.0/lifecycle

ENH Do not use deprecated lifecycle methods
silverstripe · Mar 30, 2023 · 1d6e8ce · 1d6e8ce
2 parents 2e98b60 + 20588d7
commit 1d6e8ce
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/client/dist/js/bundle.js b/client/dist/js/bundle.js
diff --git a/client/src/components/HistoryViewer/HistoryViewerVersionDetail.js b/client/src/components/HistoryViewer/HistoryViewerVersionDetail.js
@@ -10,21 +10,17 @@ import url from 'url';
 import qs from 'qs';
 
 class HistoryViewerVersionDetail extends PureComponent {
-  componentWillMount() {
+  componentDidMount() {
     this.toggleToolbarClass(true);
   }
 
   /**
    * When new props are received (from Redux dispatch events), check whether the preview
    * state changes. If so, we want to add or remove the legacy CSS modifier for the CMS
    * north toolbar based on whether the view mode is "split" (add) or anything else (remove)
-   *
-   * @param {Object} nextProps
    */
-  componentWillReceiveProps(nextProps) {
-    if (nextProps.previewState !== this.props.previewState) {
-      this.toggleToolbarClass(nextProps.previewState === 'split');
-    }
+  componentDidUpdate() {
+      this.toggleToolbarClass(this.props.previewState === 'split');
   }
 
   componentWillUnmount() {