ifNth arguments seem to be switched

[evenfrost]

handlebars-helpers/lib/comparison.js

Line 328 in 5d3405f

var isNth = utils.isNumber(a) && utils.isNumber(b) && b % a === 0;

Is there any idea behind switching arguments (b % a) here? Documentation says that

Conditionally renders a block if the remainder is zero when a operand is divided by b.

but logic seems to be inverse. I spent plenty of time trying to figure out why my ifNth is not working as supposed, or I just don't get it.

[doowb]

The following test shows an example of the usage:

handlebars-helpers/test/comparison.js

Lines 381 to 401 in 20a2fa1

	it('should render a custom class on even rows', function() {
	var source = '{{#each items}}<div{{#ifNth 2 @index}}{{else}} class="row-alternate"{{/ifNth}}>{{name}}</div>{{/each}}';
	var fn = hbs.compile(source);
	var context = {
	items: [
	{ name: 'Philip J. Fry' },
	{ name: 'Turanga Leela' },
	{ name: 'Bender Bending Rodriguez' },
	{ name: 'Amy Wong' },
	{ name: 'Hermes Conrad' }
	]
	};
	assert(fn(context), [
	'<div>Philip J. Fry</div>',
	'<div class="row-alternate">Turanga Leela</div>',
	'<div>Bender Bending Rodriguez</div>',
	'<div class="row-alternate">Amy Wong</div>',
	'<div>Hermes Conrad</div>'
	].join(''));
	});
	});

I agree that the documentation could be updated to show an example and clear up which is argument is dividing the other. Would you like to do a PR to update the code comments? I think the logic is correct because the usage in the test "feels" more natural to me.

[evenfrost]

Sure, landed here: #307.