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

After merging and splitting cells, I can't apply styles on corresponding cells

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • 3.2.5, 3.3.2, 3.4.1, 3.5
    • 3.4
    • None
    • None
    • Chrome

      On chrome

          [CMS-3680] After merging and splitting cells, I can't apply styles on corresponding cells

          Finally, there are some cases it was still happening, so there is a correction.
          The case is 2 header rows and 2 headers cols. Do merge normal cells of row 3 and 4, col 3. Split it. One is a header and one is normal...

          Raphaël Franchet added a comment - Finally, there are some cases it was still happening, so there is a correction. The case is 2 header rows and 2 headers cols. Do merge normal cells of row 3 and 4, col 3. Split it. One is a header and one is normal...

          This correction will no more be necessary (and have to be removed) because merging normal and header cellls will be forbiden by now

          Raphaël Franchet added a comment - This correction will no more be necessary (and have to be removed) because merging normal and header cellls will be forbiden by now

          The patch is not correct since you can not assume their is only one parent row: think about (that's him again) a multiselection on several rows.

          Moreover, I do not understand in your patch what happend if their is no row header... (nothing I think, that is a problem)

          Raphaël Franchet added a comment - The patch is not correct since you can not assume their is only one parent row: think about (that's him again) a multiselection on several rows. Moreover, I do not understand in your patch what happend if their is no row header... (nothing I think, that is a problem)

          This is a minor problem in a very special case, so it is best to wait for more tests and expert's checking before applying this patch

          Laurence Aumeunier added a comment - This is a minor problem in a very special case, so it is best to wait for more tests and expert's checking before applying this patch

          Laurence Aumeunier added a comment - - edited

          The split cell is a <th> cell, so after split the cells keep the <th> regardless this is header row or not

          Laurence Aumeunier added a comment - - edited The split cell is a <th> cell, so after split the cells keep the <th> regardless this is header row or not

          Laure Lopez added a comment - - edited

          Now it's the reverse :

          • Insert a table,
          • Set the first column as "Header column"
          • Merge 2 horizontal cells one of which is in the "Header column"
          • Split the cells previously merged
            > the second cell has kept the header style and there is no way to change that

          Laure Lopez added a comment - - edited Now it's the reverse : Insert a table, Set the first column as "Header column" Merge 2 horizontal cells one of which is in the "Header column" Split the cells previously merged > the second cell has kept the header style and there is no way to change that

          Note that is not an issue specific to Chrome

          Laurence Aumeunier added a comment - Note that is not an issue specific to Chrome

          This is the way (I suppose you did) to reproduce it

          • Insert a table of 3 lines (for example)
          • Set the first column has "Header column"
          • Merge two cells of the first column
          • Split the cells previously merge

          Behavior

          • The last splitted cell has no the header column style (see screenshot)

          Laurence Aumeunier added a comment - This is the way (I suppose you did) to reproduce it Insert a table of 3 lines (for example) Set the first column has "Header column" Merge two cells of the first column Split the cells previously merge Behavior The last splitted cell has no the header column style (see screenshot)

            raphael Raphaël Franchet
            llopez Laure Lopez
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Created:
              Updated:
              Resolved: