From 274d8cbc0bddcbbe4eb6d7100a4593f23454db91 Mon Sep 17 00:00:00 2001 From: Sam Date: Thu, 27 Jun 2013 16:29:31 +1000 Subject: [PATCH] if the title is the same as the value for a number, do not display it --- .../discourse/helpers/application_helpers.js | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/app/assets/javascripts/discourse/helpers/application_helpers.js b/app/assets/javascripts/discourse/helpers/application_helpers.js index 3c58e4075a9..9f380209afd 100644 --- a/app/assets/javascripts/discourse/helpers/application_helpers.js +++ b/app/assets/javascripts/discourse/helpers/application_helpers.js @@ -257,7 +257,7 @@ Ember.Handlebars.registerHelper('float', function(property, options) { @for Handlebars **/ Handlebars.registerHelper('number', function(property, options) { - var n, orig, title; + var n, orig, title, result; orig = parseInt(Ember.Handlebars.get(this, property, options), 10); if (isNaN(orig)) { orig = 0; @@ -270,10 +270,18 @@ Handlebars.registerHelper('number', function(property, options) { } // Round off the thousands to one decimal place n = orig; - if (orig > 999) { + if (orig > 999 && !options.hash.noTitle) { n = (orig / 1000).toFixed(1) + "K"; } - return new Handlebars.SafeString("" + n + ""); + + result = ""; + return new Handlebars.SafeString(result); }); /**