refactor: Change params to update_count_in_dom().

We now just pass in unread_count_elem and count to
update_count_in_dom(), and it does the work of finding
count_span and value_span now.
This commit is contained in:
Steve Howell 2017-01-15 07:24:49 -08:00 committed by Tim Abbott
parent 6045730902
commit 4597fdae27
1 changed files with 6 additions and 4 deletions

View File

@ -291,7 +291,10 @@ exports.get_stream_li = function (stream_name) {
return get_filter_li('stream', stream_name); return get_filter_li('stream', stream_name);
}; };
function update_count_in_dom(count_span, value_span, count) { function update_count_in_dom(unread_count_elem, count) {
var count_span = unread_count_elem.find('.count');
var value_span = count_span.find('.value');
if (count === 0) { if (count === 0) {
count_span.hide(); count_span.hide();
if (count_span.parent().hasClass("subscription_block")) { if (count_span.parent().hasClass("subscription_block")) {
@ -310,9 +313,8 @@ function update_count_in_dom(count_span, value_span, count) {
} }
function set_count(type, name, count) { function set_count(type, name, count) {
var count_span = get_filter_li(type, name).find('.count'); var unread_count_elem = get_filter_li(type, name);
var value_span = count_span.find('.value'); update_count_in_dom(unread_count_elem, count);
update_count_in_dom(count_span, value_span, count);
} }
function rebuild_recent_topics(stream) { function rebuild_recent_topics(stream) {