I've tried to generalise the code so that it can handle things like row heights without using hacks.
I've added quite a lot of different tests of the new code to make sure it performs as expected and gives the same results as previous iterations of the code (except for where new features have been implemented).
I've also tried to clarify our use of variable names to make the distinction between openxlsx style, our xltabr styles, and the style string we're using. Hoepfully this will make it a bit easier to update in future.
Finally, I've added support for text colour and font inheritence. I think this might not have been implemented before?
I've tried to generalise the code so that it can handle things like row heights without using hacks.
I've added quite a lot of different tests of the new code to make sure it performs as expected and gives the same results as previous iterations of the code (except for where new features have been implemented).
I've also tried to clarify our use of variable names to make the distinction between openxlsx style, our xltabr styles, and the style string we're using. Hoepfully this will make it a bit easier to update in future.
Finally, I've added support for text colour and font inheritence. I think this might not have been implemented before?