fillColor turn's black when specifing an object to lineWidth

[ferflores507]

This happens when you assign the object with some or all properties (top, right, bottom, left).

The workaround is to assign fillColor a value that is not false or null (another issue), in my case I had to assign white.

[JavorovNetIt]

I have the same issue!

[MikeDabrowski]

Me too!

@simonbengtsson

jsPDF-AutoTable/src/tableDrawer.ts

Lines 402 to 409 in 896a73d

	drawBorderForSide(
	doc,
	cell,
	cursor,
	side,
	fillStyle || 'S',
	lineWidth[side]
	)

Here is the place where the fillColor is replaced by 'S' which is the default for line stroke/fill. Imo, lines use jsPDF.setDrawColor, don't they?

I can create PR with the change.

[simonbengtsson]

Would be great! If all the examples works and the tests pass I'll merge right away 👌

[MikeDabrowski]

So the changes would be:
Remove fallback here:

jsPDF-AutoTable/src/tableDrawer.ts

Line 407 in 896a73d

fillStyle || 'S',

And replace 'F' with cellStyles.fillColor (casted to null) here:

jsPDF-AutoTable/src/tableDrawer.ts

Line 395 in 896a73d

doc.rect(cell.x, cursor.y, cell.width, cell.height, 'F')

I actually doubt this 395 line is needed. I'm testing using my project's table where I use 'plain' theme, no borders here.
I'll get to work on the PR

[MikeDabrowski]

@simonbengtsson PR ready, I'd appreciate if you take a look