Skip to content

E
edx-platform
Commit 1927551a by Tyler Hallada
Options

Re-render XBlockOutlineView on highlights change

parent 43d6cc20
Showing with 6 additions and 1 deletions
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 ...@@ -635,6 +635,7 @@ define(['jquery', 'edx-ui-toolkit/js/utils/spec-helpers/ajax-helpers', 'common/j
saveHighlights(); saveHighlights();
expectServerHandshakeWithHighlights(updatedHighlights); expectServerHandshakeWithHighlights(updatedHighlights);
expectHighlightLinkNumberToBe(updatedHighlights.length);
openHighlights(); openHighlights();
expectHighlightsToBe(updatedHighlights); expectHighlightsToBe(updatedHighlights);
......
cms/static/js/views/xblock_outline.js
...@@ -224,7 +224,11 @@ define(['jquery', 'underscore', 'gettext', 'js/views/baseview', 'common/js/compo ...@@ -224,7 +224,11 @@ define(['jquery', 'underscore', 'gettext', 'js/views/baseview', 'common/js/compo
}, },
onSync: function(event) { 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(); this.onXBlockChange();
} }
}, },
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment