diff --git a/frontend_tests/node_tests/stream_events.js b/frontend_tests/node_tests/stream_events.js index a2c7336a58..4c961aa67c 100644 --- a/frontend_tests/node_tests/stream_events.js +++ b/frontend_tests/node_tests/stream_events.js @@ -124,7 +124,7 @@ run_test('update_property', () => { with_overrides(function (override) { override('stream_list.refresh_pinned_or_unpinned_stream', noop); stream_events.update_property(1, 'pin_to_top', true); - checkbox = $('#pinstream-1'); + checkbox = $(".subscription_settings[data-stream-id='1'] #sub_pin_to_top_setting .sub_setting_control"); assert.equal(checkbox.prop('checked'), true); }); diff --git a/static/js/stream_events.js b/static/js/stream_events.js index d00808a0a0..d48e157c35 100644 --- a/static/js/stream_events.js +++ b/static/js/stream_events.js @@ -31,7 +31,7 @@ function update_stream_email_notifications(sub, value) { } function update_stream_pin(sub, value) { - var pin_checkbox = $('#pinstream-' + sub.stream_id); + var pin_checkbox = $(".subscription_settings[data-stream-id='" + sub.stream_id + "'] #sub_pin_to_top_setting .sub_setting_control"); pin_checkbox.prop('checked', value); sub.pin_to_top = value; } diff --git a/static/templates/subscription_settings.handlebars b/static/templates/subscription_settings.handlebars index 16e5c8e845..7c8d2aad5e 100644 --- a/static/templates/subscription_settings.handlebars +++ b/static/templates/subscription_settings.handlebars @@ -50,7 +50,7 @@