Log function does not limit log size

[JefStat]

I replaced most of my tail windows with this box log. I ran into a problem were I would into huge slowdowns because of a the large amount of log lines. I had to put in a way to limit the log html.

if (box.logDiv.childElementCount > 500) {
	box.logDiv.replaceChildren(...Array.from(box.logDiv.children).slice(-100))
}

More specifically this is how I use the box api

export const boxTailSingleton = (ns, title, icon, height) => {
	var res = [];
	sidebar.querySelectorAll('div.sbitem').forEach(sbitem => res.push({ sbitem, title: sbitem.querySelector('div.head > span').innerText }));
	let box = res.find(o => o.title === title);
	if (box) {
		box = box.sbitem;
	} else {
		box = createSidebarItem(title, "<div/>", icon);
	}
	if (height) box.style.height = height;

	const _print = ns.print;
	ns.print = (m) => {
		box.log(`<span>${m}</span>`);
		_print(m);
		box.logDiv = box.body.querySelector('div.log');
		if (box.logDiv.childElementCount > 500) {
			box.logDiv.replaceChildren(...Array.from(box.logDiv.children).slice(-100))
		}
	}
}

[Snarling]

Good point.

I think the cleanest and probably best performance way to limit log entries would be
while (box.logDiv.children.length > logEntryLimit) box.logDiv.children[0].remove();

I'm torn on whether to include a limiter by default, for most of my use cases I want to preserve the full record of what the script has done, but none of my scripts make an obscene number of log entries either. Maybe I will include a limiter but set the default limit to infinity.

[Snarling]

I'm not going to add a limiter by default, but I'll leave this issue here for those who do want to implement a log entry limiter.