Add more configuration for heading in INSERT_TABLE_COMMAND

[Sebastien-Ahkrin]

The idea behind this changement is to allow the user of the API to choice if the header will be only row, column or Both.

For example:

function onClick() {
  activeEditor.dispatchCommand(INSERT_TABLE_COMMAND, {
    columns,
    includeHeaders: true,
    rows,
  });
}

This will make the header as column and row like before.

but, if you add columns and row as an object for includeHeaders like that:

function onClick() {
  activeEditor.dispatchCommand(INSERT_TABLE_COMMAND, {
    columns,
    includeHeaders: {columns: false, rows: true},
    rows,
  });
}

Or like that if you make columns as true and rows as false

Do this functionnality looks good for you ?

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated
lexical	✅ Ready (Inspect)	Visit Preview	💬 Add your feedback	Feb 7, 2023 at 4:15PM (UTC)
lexical-playground	✅ Ready (Inspect)	Visit Preview	💬 Add your feedback	Feb 7, 2023 at 4:15PM (UTC)

[thegreatercurve]

The semantics might be slightly odd as you'd always expect includeHeaders to be a boolean, but it's probably not worth introducing a breaking change over.

[Sebastien-Ahkrin]

The semantics might be slightly odd as you'd always expect includeHeaders to be a boolean, but it's probably not worth introducing a breaking change over.

yeah, i choosed to only add a type with "boolean" included. becausei think it would be bad for every user to fix the breaking changes. Tell me if i have to change something ! :)