From 1927551ac6d6cea1a40ce877e8e765e9752e5251 Mon Sep 17 00:00:00 2001 From: Tyler Hallada Date: Wed, 8 Nov 2017 13:29:44 -0500 Subject: [PATCH] Re-render XBlockOutlineView on highlights change --- cms/static/js/spec/views/pages/course_outline_spec.js | 1 + cms/static/js/views/xblock_outline.js | 6 +++++- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/cms/static/js/spec/views/pages/course_outline_spec.js b/cms/static/js/spec/views/pages/course_outline_spec.js index a7a3c2639a..9800a3261b 100644 --- a/cms/static/js/spec/views/pages/course_outline_spec.js +++ b/cms/static/js/spec/views/pages/course_outline_spec.js @@ -635,6 +635,7 @@ define(['jquery', 'edx-ui-toolkit/js/utils/spec-helpers/ajax-helpers', 'common/j saveHighlights(); expectServerHandshakeWithHighlights(updatedHighlights); + expectHighlightLinkNumberToBe(updatedHighlights.length); openHighlights(); expectHighlightsToBe(updatedHighlights); diff --git a/cms/static/js/views/xblock_outline.js b/cms/static/js/views/xblock_outline.js index 578ea70218..28312802ca 100644 --- a/cms/static/js/views/xblock_outline.js +++ b/cms/static/js/views/xblock_outline.js @@ -224,7 +224,11 @@ define(['jquery', 'underscore', 'gettext', 'js/views/baseview', 'common/js/compo }, onSync: function(event) { - if (ViewUtils.hasChangedAttributes(this.model, ['visibility_state', 'child_info', 'display_name'])) { + var hasChangedAttributes = ViewUtils.hasChangedAttributes( + this.model, + ['visibility_state', 'child_info', 'display_name', 'highlights'] + ); + if (hasChangedAttributes) { this.onXBlockChange(); } },