[puppeteer] Make height/width styles optional

[ivan-jelev]

Hello,

We are trying to migrate from exporter 2.x to 3.x-beta + Highchart 11+ and facing couple of issues.
One of them is unwanted container height/width CSS which doesn't reflect runtime changes to chart width/height.

Use case: we have horizontal line chart with dynamic amount of horizontal columns/bars. Based on the amount of bars we are calculating the total height of the chart in the callback function.

Due to this css styles we are getting our chart cut (I assume scrolling appears on real page).

Plz consider to make this styles optional or maybe completely remove.

Note that it worked as expected in exporter v2.x
Plz let me know if you need more info.

Thanks.

node-export-server/templates/template.html

Lines 190 to 215 in fd0e4ed

	let injCSS = false;
	if (exportOptions.export.strInj) {
	const injCSS = document.createElement('style');
	injCSS.textContent = `
	#chart-container {
	height: ${chart.height}px;
	max-width: ${chart.width}px;
	}
	#chart-container #container{
	height: ${chart.height}px;
	max-width: ${chart.width}px;
	}
	`;
	document.head.appendChild(injCSS);
	}
	createChart(chartOptions, exportOptions);
	// Remove the injected CSS
	if (injCSS) {
	document.head.removeChild(injCSS);
	}

[ivan-jelev]

Here is an example of the chart we are trying to export:

https://jsfiddle.net/ivanjelev/4qp0Lco9/

[jszuminski]

Thanks for reporting @ivan-jelev!

Currently, we're working on a few prioritized issues related to server health (which is crucial for us to solve asap), but I'm adding this issue to our backlog for further development.

I'll keep you posted here.

PS. In the meantime, as a workaround, you can modify this code in template.html to fit your particular case.

[PaulDalek]

Fixed by #440.