Calculate multiline text cell width and add nowrap option

[J-F-Liu]

No description provided.

[packetmonkey]

So far this looks good. Can you please add a couple of specs to ensure that the behavior with and without nowrap will continue to work correctly?

Thanks!

[andyjeffries]

@pointlessone is there any reason this can't be merged? (tagging you because you seem to be the person who commits most here).

I really need this functionality for my project. I can use a fork, but I'd rather help (if needed) test this fix and get it in the mainline.

[gettalong]

@andyjeffries When you look at the code of the pull request, you can see that there are no tests and no documentation. So once someone has the time to add those and make sure the current behaviour still works as before, the maintainers can have another look at it.

[andyjeffries]

Thanks @gettalong, I don't know the codebase but I'm trying to get in to this.

I've just checked current master though and it fails for me:

11:07 ~/Developer/vendor/prawn-table (master ✗) $ git pull -r
Already up to date.
11:07 ~/Developer/vendor/prawn-table (master ✗) $ bundle exec rspec
Prawn specs: Running on Ruby Version: 3.0.4
........................................................................F............................................................................................................................................................

Failures:

  1) Prawn::Table Prints table on one page when using subtable with colspan > 1
     Failure/Error: expect(pdf.page_count).to eq 1

       expected: 1
            got: 2

       (compared using ==)
     # ./spec/table_spec.rb:1322:in `block (2 levels) in <top (required)>'

Finished in 0.91638 seconds (files took 0.12048 seconds to load)
229 examples, 1 failure

Failed examples:

rspec ./spec/table_spec.rb:1299 # Prawn::Table Prints table on one page when using subtable with colspan > 1

So I'll ignore that one spec failure and work on making sure the other current specs pass and new ones are added for this feature.