Text: Add Markdown table support

[Isvvc]

Is your feature request related to a problem? Please describe.
Yes. Documents with Markdown tables do not have their tables rendered.

Describe the solution you'd like
Standard Markdown table rendering.

Describe alternatives you've considered
As developing a UI for making tables could be difficult, simply using the standard Markdown to HTML table conversion would be enough.

[juliushaertl]

Yes, we should look into implementing support for https://github.github.com/gfm/#tables-extension-

[jancborchardt]

@Isvvc is this something you would like to help contribute? 😊 The Text app is of course at the first version, and any contribution is welcome – we’ll help you get set up if you need anything.

[Isvvc]

@jancborchardt I could definitely make an attempt but am not sure exactly where to start. I have experience with text manipulation but none with Nextcloud apps or Vue.js.

[jancborchardt]

@Isvvc cool! Maybe @juliushaertl or someone from @nextcloud/vue has some pointers to help. :)

[ChristophWurst]

For Vue I suggest to read the Vue docs and then some code from Nextcloud apps. This will make you understand how we use it here.

---

someone from @nextcloud/vue has some pointers to help. :)

I would kindly ask to not abuse the team for stuff like this. This just floods everyone's notifications. The maintainers of this app should be able to help.

Unsubscribed. Please @mention me directly if you need any specific help.

[juliushaertl]

Regarding the integration tables, I would recommend to first have a look at the tiptap table nodes to see what they support and how/if we can represent that in github flavored markdown. Since most likely not all will be supported, we need to create our own table nodes or inherit from the tiptap ones and add them to https://github.com/nextcloud/text/blob/master/src/EditorFactory.js#L45-L59

Here are some rough steps with code hints:

• Make sure markdownit actually renders the tables: https://github.com/nextcloud/text/blob/master/src/EditorFactory.js#L73-L74
• Make sure node definitions properly parse the rendered output to the table node types
• Implement tiptap to markdown conversion in custom nodes https://github.com/nextcloud/text/blob/master/src/marks/index.js#L46-L75

[elpraga]

Is there a roadmap as approximately when this functionality could be implemented?

[juliushaertl]

No, there is no estimation that we can give. Of course contributions are always welcome.

[elpraga]

@juliushaertl I wish i were able to contribute with the code. I'm an English teacher, not a coder. Even though I did have a look at tiptap tables (they work great BTW), I was not able to make them work locally.

I'm grateful that nextcloud exists and gives us the freedom to host files at home instead of having to rely on a big corporation. I do file bug reports and try to contribute with suggestions, but, unfortunately, I cannot contribute with code.

Thank you for your work, and if tables in the text app could be implemented, it would be much appreciated.

[juliushaertl]

@elpraga Thank you very much the kind words. Keep your contributions coming :wink:

[jonathandeclan]

Since this uses tiptap, I don't understand why we don't have tables
https://tiptap.scrumpy.io/tables

[guddl]

Since this uses tiptap, I don't understand why we don't have tables
https://tiptap.scrumpy.io/tables

Yes please :-)

[juliushaertl]

@jonathandeclan @guddl Please read https://github.com/nextcloud/text/issues/114#issuecomment-507180208 and use Github Reactions feature instead of commenting that also need a specific feature. That way we can see your support without notifying all users subscribed to a given issue without adding anything new to the discussion.

TLDR: As tiptap doesn't have markdown support out of the box, additional implementation work is required. Contributions are always welcome.

[jakobroehrl]

It would be awesome if the tables could be sorted and are filterable :)

[markuskramerIgitt]

I happily used markdown tables and assumed there were a standard feature of Nextcloud.

After an upgrade from Nextcloud 16 to 17, my markdown tables are no longer rendered as html table.

Can someone confirm that Nextcloud 16 renders markdown tables?

I resolved my markdown table support need via the Pico CMS app.

Therefore I no longer require this enhancement.

[davidbe]

It would be nice to hav an implementation like this: https://blog.documentnode.io/when-markdown-meets-tables

[fajabird]

What about an option to switch to raw markdown editing so any formatting supported by the rendering engine can be used even when not supported by the text-editing UI. This of course would require the support of extended markdown syntax by the engine.

I think as this editor used markdown in the backend there should always be an option to switch to RAW mode and supported elements should be rendered as expected. E.g. when I copy paste an markdown file to my nextcloud foldert and open it with Text it should display all elements correctly, not only those that are supported by the UI. For the time beeing those not supported by the UI could be marked so its obvious that those can only be edited from the RAW mode.

[juliushaertl]

@fajabird See #123

[asoltesz]

This is a pretty big blocker for any realistic use of PicoCMS with NextCloud since any moderately complex page will contain tables.

[Daksol]

Version of Markdown in Mattermost would be useful, it has Tables.