Port Table edit plugin

[Andres-CT98]

Plugin needs

Function	Need from DOM	Cursor position	Is Drag and Drop?
Mover (Selector)	Table position on Document	Always outside table	No
Resizer	Table position on Document	Always outside table	Yes
Inserter	Table and cell position on Document	Always outside table	No
Cell resizer	Table and cell position on Document	Always inside table	Yes

Progress

• Copy plugin to Content model
  • The pluginUtils Disposable, DragAndDropHandler, and DragAndDropHelper were copied too Port plugin and roosterjs-editor-dom utilities #2388
• Change IEditor to IStandaloneEditor in plugin and make necessary adaptations
  • TableResize renamed to TableEditPlugin
  • TableSelector renamed to TableMover
• Make plugin usable on Demo site
  • Old plugin is removed from demo
• Change functions to use already existing Content model ones where applicable
  • Table Mover and Table Inserter now use CM fully on their functionality
  • The handles of the all plugins still only use js and css
  • Table resizer and Cell resizer use CM on its functionality of resizing and use js for writeback
• Bring to Standalone Editor needed functions
  • createElement and Rect related plugin utilities were added Port plugin and roosterjs-editor-dom utilities #2388
  • isMobileOrTablet added to core, getScrollContainer and getVisibleViewport added to editor Restore getScrollContainer, getVisibleViewport, and IsMobileOrTablet #2400
  • Modified insertTableRow/Column to select any new insertion. Select newly inserted row/column #2412
  • Addition of getDomStyle to Dom Helper Add getDomStyle to DOMHelper #2419
• Remove all uses of non Content model dependencies
  • No plugin has dependencies outside content model packages
  • onShowHelperElement method removed
  • Functionality to resize all columns equally while pressing shift removed, as it was non functioning and unused.
  • Normalization of cells occurs only in normalizeTable
  • A check for Min Width was used in CellResizer when resizing columns due to the possibility of normalisation causing a wrong writeback
• Add test cases
  • Port and rework of unit tests for Table edit plugin. Scenarios tested:
    • Table Edit Plugin:
      • Dismiss table editor on mouse out
      • Do not dismiss table editor on mouse out, related target is contained in scroll container
      • Do not dismiss table editor on mouse out, table editor not
      • Do not dismiss table editor on mouse out, related target null
      • Do not dismiss table editor on mouse out, currentTarget null
    • Table Mover
      • Display component on mouse move inside table
      • Display component, Top of table is visible in the scroll container scrolled down
      • Do not display component, top of table is no visible in the container
      • Do not display component, Top of table is no visible in the scroll container
      • On Click
    • Cell Resizer
      • Increase the height of the first row of a standard table
      • Decrease the height of the first row of a standard table
      • Increase the height of the last row of a standard table
      • Decrease the height of the last row of a standard table
      • Increase the width of the first column of a standard table
      • Decrease the width of the first column of a standard table
      • Increase the width of the last column of a standard table
      • Decrease the width of the last column of a standard table
    • Table Resizer
      • Removes table resizer on input
      • Removes table resizer on content change
      • Removes table resizer on scrolling
      • Increases the width of a standard table
      • Increases the height of a standard table
      • Increases the width and height of a standard table
      • Decreases the width of a standard table
      • Decreases the height of a standard table
      • Decreases the width and height of a standard table
    • Table Inserter
      • Adds a new column if the vertical inserter is detected and clicked
      • Adds a new row if the horizontal inserter is detected and clicked
      • Does not add inserter if top left corner hovered
      • Does not add new column if top middle clicked on merged top row
      • Does not add new row if left middle clicked on merged first column

Warnings and implications

• An issue was found when changing zoom: The CSS of the table not reflecting the values of that table in Content Model Table widths.

[juliaroldi]

Should we have the test cases in this PR to ensure this plugin behaves as expected?

[Andres-CT98]

Should we have the test cases in this PR to ensure this plugin behaves as expected?

I prefer it be done in a separate one to keep the focus of this PR on the code of the implementation.

[juliaroldi]

Should we have the test cases in this PR to ensure this plugin behaves as expected?

I prefer it be done in a separate one to keep the focus of this PR on the code of the implementation.

I'm worried that if someone makes a change that could potentially break this feature, it might go unnoticed until the feature breaks, because we do not have unit test. But if you already have a PR for unit test ready, I think it is ok the way it is.

[JiuqingSong]

The dev code change looks good to me now.