Required control words for RTF Table

[SheetJSDev]

First off, awesome work! Glad to see our codepage library was helpful :)

https://jsfiddle.net/omyjg5a9/ generates an RTF table that Word 2016 and OSX TextEdit recognize as a borderless table. rtf.js doesn't seem to recognize this as a table :(

The generator is extremely simple, using a handful of control words for the table block. Here's an example with some comments prefixed by #:

{\rtf1\ansi
  \trowd\trautofit1 # start of row
  \cellx1\cellx2    # Word seems to need this, even though the widths are not used
  \pard\intbl       # mark before cells
    a\cell          # each cell is terminated with the \cell control word
    b\cell
  \pard\intbl\row   # end of row

  \trowd\trautofit1
  \cellx1\cellx2
  \pard\intbl
    1\cell
    2\cell
  \pard\intbl\row
}

A request for table support was already raised in #15 . Based on your understanding of the specification, do those control words suffice to represent a table? If so, would you accept a PR for table support based on those words?

[zoehneto]

The rtf example you provide seems sufficient for a minimal table (cellx is a required part of the table definition tbldef according to the spec).

We would certainly appreciate a PR for (minimal) table support, we can always later expand it to include all the additional formatting that is possible. @tbluemel originally started implementing this in the branch table-support (including some major rearchitecting to make it work), but that never reached the stability or feature completeness of the current version, none the less it might be interesting to look at that branch for some pointers on implementing it.

[zoehneto]

If you are still interested, I have just ported the table-support branch to the current master so it is now all typescript and has the test suite, which should make understanding and development a lot easier. You can find it in the branch table-support-typescript.

[tbluemel]

Nice. I'm still interested, I just don't have the time to work on it currently.

[didiergm]

Just wondering if you had time to look into this pr ? I just came across this awesome piece of work and was wondering if/when table support will be included

[tbluemel]

Unfortunately, I haven't really had time to work on this project recently.

[andyr-doctolib]

Is there a plan to rebase again table-support-typescript on master in order to have minimal support for tables?

The lib is great and works well for most of my use cases but the lack of table support is kind of a deal breaker atm

[davidkroukamp]

Is there any reason why this as never added to master?

I ask because we use 3.0.7 and no table support has become an issue.

I am attempting to get the table-support-typescript branch updated to the latest and changes around RtfDestination.ts and Renderer.ts become unclear which are made as a consequence of the table support and which are updates from the simple fact that the branch is 4 years old @zoehneto @tbluemel

Some validation for myself are needed on the below:

• Seems like Renderer.ts needs to change to use RenderElement | RenderContainer and not the IContainer element
• There seems to be a few references to using Chp and Pap directly and not use RenderChp.ts and RenderPap.ts in RtfDestination.ts
• Also in RtfDestination.ts it seems that _addFormatIns is swapped out for _updateFormatIns, which uses some propertyChanged stuff and a flush method
• RenderElements.ts used JQuery I've changed for HtmlElements and added Helper.addStyle in place of JQuery#css()

Is all the above still needed for Table support or could some stuff be made easier?

Any help will be appreciated!

[zoehneto]

As stated above the branch never got to a state where it could actually parse anything but simple examples (and not in the 'some things arent't handled yet' but in the 'it crashes randomly' way). That is why it was never merged.

[Miniontoby]

Hello, can the table-support-typescript branch by any chance be updated to the latest version of the master branch?

This is literally the only project that allows you to parse rtf in javascript without having to pay.

I am able to install the current table-support-typescript branch into my project, but it seems a little outdated and it misses the actual typescript typings and still requires JQuery which is the actual problem atm since that makes it not work. Even when I install JQuery it just doesn't want to work:

ReferenceError: $ is not defined
Error
    at new RTFJSError (http://localhost:1420/node_modules/.vite/deps/chunk-DN6QSDXT.js?v=e56afb5f:14054:28)
    at http://localhost:1420/node_modules/.vite/deps/chunk-DN6QSDXT.js?v=e56afb5f:13990:27
    at async ViewDocText (http://localhost:1420/src/routes/+page.svelte?t=1735831177558:1064:6)

I guess that it is because $ is not a global variable, and it isn't imported by the rtf.js code for some reason...

I could fix this by using:

export {};
declare global {
	var $: typeof JQuery;
}
// @ts-ignore JQuery has no typedefinion
import JQuery from 'jquery';
globalThis.$ = JQuery;

But as you might understand, this is just a stupid workaround and is not something I would actually want to use in my code.

I understand the version might crash randomly, but that's fine with me.
All I need is someone to just update the code to match the master branch and then I'm fine.