the composer will now count only significant whitespaces

This commit is contained in:
Régis Hanol 2013-03-12 23:54:29 +01:00
parent cde9d8c3a9
commit 439ac95b73
2 changed files with 9 additions and 3 deletions

View File

@ -478,7 +478,7 @@ Discourse.Composer = Discourse.Model.extend({
},
/**
Computes the length of the reply minus the quote(s).
Computes the length of the reply minus the quote(s) and non-significant whitespaces
@property replyLength
**/
@ -486,7 +486,7 @@ Discourse.Composer = Discourse.Model.extend({
var reply = this.get('reply');
if(!reply) reply = "";
while (Discourse.BBCode.QUOTE_REGEXP.test(reply)) { reply = reply.replace(Discourse.BBCode.QUOTE_REGEXP, ""); }
return reply.trim().length;
return reply.replace(/\s+/img, " ").trim().length;
}.property('reply')
});

View File

@ -10,10 +10,16 @@ describe("Discourse.Composer", function() {
});
it("trims whitespaces", function() {
var composer = Discourse.Composer.create({ reply: "\nbasic reply\t" });
var composer = Discourse.Composer.create({ reply: " \nbasic reply\t" });
expect(composer.get('replyLength')).toBe(11);
});
it("count only significant whitespaces", function() {
// this will count the '\n' only once
var composer = Discourse.Composer.create({ reply: "ba sic\n\nreply" });
expect(composer.get('replyLength')).toBe(12);
});
it("removes quotes", function() {
var composer = Discourse.Composer.create({ reply: "1[quote=]not counted[/quote]2[quote=]at all[/quote]3" });
expect(composer.get('replyLength')).toBe(3);