Uploaded image for project: 'CMS'
  1. CMS
  2. CMS-3793

Removing a merged cell in table edition leads to a corrupted table

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • 4.2.0
    • 3.2, 3.3, 3.4, 3.5
    • Inline Editor
    • None

      Create a table of 5 rows and 5 columns
      Merge verticaly 3 cells in the middle
      Enter text in each cell to visualize it
      Remove line 2

      The cells are missplaced (missing cells on some lines)

          [CMS-3793] Removing a merged cell in table edition leads to a corrupted table

          fine in v4

          Raphaël Franchet added a comment - fine in v4

          We can try to upgrade before trying to fix it by ourselves (or open it on tinymce)

          Raphaël Franchet added a comment - We can try to upgrade before trying to fix it by ourselves (or open it on tinymce)

          Note that removing line 1, 3, 4 or 5 will work fine.

          Solutions are :

          • forbid removing lines with merged cells (too bad, especially if a multiselection will wrap the merged cell)
          • remove merged cell, and create 2 empty cells on line 3 and 4
          • remove merged cell, and create 1 empty cell merged on line 3 and 4
          • move merge cell to line 3 (and reduce its scope to line 4)

          I think the only solution that is acceptable for the final user is the last one. Because if the content of the cell is bottom align... it would be non-understandable for him to loos its data.

          Raphaël Franchet added a comment - Note that removing line 1, 3, 4 or 5 will work fine. Solutions are : forbid removing lines with merged cells (too bad, especially if a multiselection will wrap the merged cell) remove merged cell, and create 2 empty cells on line 3 and 4 remove merged cell, and create 1 empty cell merged on line 3 and 4 move merge cell to line 3 (and reduce its scope to line 4) I think the only solution that is acceptable for the final user is the last one. Because if the content of the cell is bottom align... it would be non-understandable for him to loos its data.

          (blue cross stands for missing cells)

          Raphaël Franchet added a comment - (blue cross stands for missing cells)

            raphael Raphaël Franchet
            raphael Raphaël Franchet
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: