jquery.sparkline icon indicating copy to clipboard operation
jquery.sparkline copied to clipboard

Formatting of values in the range -100 to -999 show an extra comma

Open gmathijssen opened this issue 10 years ago • 3 comments

See http://jsfiddle.net/gmathijssen/PfzXg/731/

Hoover over -105 and you will see -,105 instead of -105 This can be fixed in the formatNumber() function

gmathijssen avatar Apr 09 '14 10:04 gmathijssen

+1

jkieley avatar Oct 20 '15 17:10 jkieley

Same issue for -123123, -123123123 and any negative 3n-digit numbers.

AvaelKross avatar Sep 01 '16 12:09 AvaelKross

Here is a fixed function:

formatNumber = function (num, prec, groupsize, groupsep, decsep) {
    var p, i;
    num = (prec === false ? parseFloat(num).toString() : num.toFixed(prec)).split('');
    p = (p = $.inArray('.', num)) < 0 ? num.length : p;
    if (p < num.length) {
        num[p] = decsep;
    }
    shift = num[0] == '-' ? 1 : 0;
    for (i = p - groupsize; i > shift; i -= groupsize) {
        num.splice(i, 0, groupsep);
    }
    return num.join('');
};

AvaelKross avatar Sep 01 '16 13:09 AvaelKross