Skip to content

Feature quill table#566

Open
kyo6 wants to merge 6 commits intoNodeBB:masterfrom
kyo6:feature-quill-table
Open

Feature quill table#566
kyo6 wants to merge 6 commits intoNodeBB:masterfrom
kyo6:feature-quill-table

Conversation

@kyo6
Copy link
Copy Markdown
Contributor

@kyo6 kyo6 commented Apr 9, 2026

Add Table Support to Quill Editor

Changes

This update introduces full table editing support to the Quill editor plugin for NodeBB, including:

  • Table Toolbar

    • Added table-related action buttons to the editor (insert table, delete table, add/remove rows and columns, merge cells, etc.)
  • Styling Improvements

    • Support for both light and dark themes
    • Improved table border visibility (default border width adjusted from 0.14 to 0.18)
    • Added adaptive row height and cell selection styles
  • Feature Integration

    • Integrated quill-table.js module via plugin.json
    • Updated migrator.js to support parsing table content in Delta format
    • Added table screenshots for documentation
  • Compatibility

    • Fully compatible with the existing Quill editor setup

Modified Files

  • static/lib/quill-nbb.js – Added table toolbar configuration
  • static/scss/post.scss – Added table-related styles
  • lib/migrator.js – Integrated table parser
  • plugin.json – Included quill-table module
  • package.json – Version bump
  • Added two screenshots demonstrating table functionality

Testing Suggestions

  • Test table insertion and editing within the editor
  • Verify rendering in both light and dark themes
  • Validate saving and rendering of table content

@julianlam
Copy link
Copy Markdown
Member

I see this is quill 1 table... merging this would make it harder to later upgrade to Quill v2 (not that there are any plans to, but just wondering...)

@kyo6
Copy link
Copy Markdown
Contributor Author

kyo6 commented Apr 9, 2026

I noticed that the project is currently using Quill 1, so I chose the Quill 1 table module. Do you have plans to upgrade to Quill 2 and support the Quill 2 table module?

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.

2 participants