search

vandeseer / easytable

Small table drawing library built upon Apache PDFBox
MIT License
246 stars 94 forks source link

Wrong linebreaking #81

Closed AndreKoepke closed 4 years ago

AndreKoepke commented 4 years ago

Hi @AndreKoepke,

there's still an issue here. Didn't see that the first time unfortunately. Please have a look again at the "excelLike.pdf" file: In the last line on the cell on the right, the line should not be justified and what is of more importance: the second to last cell in the middle has a strange line breaking – just compare this with the image in the README file.

I am currently working on an automatic comparison of the generated PDF files so that such differences will be detected early on and not only by having a look manually ... As soon as I'm done I will put all the generated files from the latest version before this change under version control and we will hopefully see that there's a change in the rendered PDF documents.

Best, Stefan

Originally posted by @vandeseer in https://github.com/vandeseer/easytable/pull/79#issuecomment-599201065

AndreKoepke commented 4 years ago

I found the error. If there is only a dot or something as first char, then it breaks.

AndreKoepke commented 4 years ago

And found the other bug. With my new wordbreak-algo it shows up. Fix is on the way.

AndreKoepke commented 4 years ago

First bug: image

Second bug: image

AndreKoepke commented 4 years ago

Now there is something wrong with the cells-background. But I have no idea what's going wrong. @vandeseer can you have a look? image

(I wrote it in a old-version of easytable, because we are using the old version in a project)

vandeseer commented 4 years ago

Thanks for your effort @AndreKoepke!

I changed the files accordingly on the current develop branch. Also did I check the issue with the background: It doesn't happen on the most recent version, so that's fine :+1: :smiley:

I will close this issue as soon as I have released the new version.

vandeseer commented 4 years ago

Version 0.6.3 is published now.