Skip to content

feat: add full HTML table supprt using ProseMirror - #610

Open
osaydayoub wants to merge 2 commits into
sibiraj-s:masterfrom
osaydayoub:feature/add-table-support
Open

feat: add full HTML table supprt using ProseMirror#610
osaydayoub wants to merge 2 commits into
sibiraj-s:masterfrom
osaydayoub:feature/add-table-support

Conversation

@osaydayoub

Copy link
Copy Markdown

This PR contains:

  • bugfix
  • feature
  • refactor
  • tests
  • documentation
  • others

Breaking Changes?

  • yes
  • no

Checklist

  • commit messages follow the Conventional Commits pattern
    • A feature commit message is prefixed "feat:"
    • A bugfix commit message is prefixed "fix:"
    • A document related commit is prefixed "docs:"
  • docs have been added / updated (for bug fixes / features)

Describe Your Changes

Hi, I'm Osayd, a computer science student in my final semester. As part of my industrial project with Dell Technologies,
I was tasked with adding full HTML table support to ngx-editor, which is used by Dell’s internal documentation sites.

This feature includes:

  • A new Table menu button and dropdown for table editing

  • Real table insertion using prosemirror-tables

  • Table resizing, keyboard navigation (Tab), and other plugins

  • All files organized to align with the existing module structure

  • I previously shared this plan in this comment on May 6, but haven’t received a reply yet. Now that the feature is ready and working locally, I’d like to open this PR for your review.

  • Demo video: https://youtu.be/CkVnP75ouOk

I’d be happy if this feature could be added to the project. Let me know if I need to make any changes to better follow the project’s contribution guidelines — including tests, commit messages, documentation, or anything else.

Thanks !

Osayd

Does this PR affects any existing issues?

  • yes
  • no

@joehansen

Copy link
Copy Markdown

Hey @osaydayoub thanks for your work, we also need this.

@sibiraj-s Is there a chance that this feature will be available in the near future? Thanks

@osaydayoub

Copy link
Copy Markdown
Author

Hi @sibiraj-s , just checking in to see if there are any updates on this PR. Happy to make adjustments if needed to get it merged. Thanks!

@sibiraj-s

Copy link
Copy Markdown
Owner

Apologies. got busy with work. Will take a look this weekend and publish if everything is fine.

@Hulcsa

Hulcsa commented Oct 10, 2025

Copy link
Copy Markdown

+1

@pbabala

pbabala commented Oct 21, 2025

Copy link
Copy Markdown

Hey @osaydayoub thanks for your work, we also need this.

@sibiraj-s Is there a chance that this feature will be available in the near future? Thanks

+1

@markoberholzer-es

Copy link
Copy Markdown

+1

@RANSoftSS

Copy link
Copy Markdown

@sibiraj-s Is there any chance to get this in the next version? We would really need the table functionallity ^^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants