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

Problem when creating square list on pasted text

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • 3.1.1, 3.2
    • 3.1
    • Inline Editor
    • None

      The problem seem to come from <br/> in <p> : test with this markup :

      <p>Header <br /> First list element <br />Second list element<br /> third list element<br /> footer</p>

      Try creating a square list from "First list element" to "third list element" included.

      The result markup is :

      <p>Header</p>
      <ul class="disc">
      <li> First list element </li>
      <li>Second list element</li>
      <li> third list element</li>
      </ul>
      <p>footer</p>

      but should be :

      <p>Header</p>
      <ul class="square">
      <li> First list element </li>
      <li>Second list element</li>
      <li> third list element</li>
      </ul>
      <p>footer</p>

          [CMS-2483] Problem when creating square list on pasted text

          First of all, please note this is not the right way to make lists (The proof is that your way works only if the last line you select is not the last line in the para)
          Secondly, it is often a better thing to partially select lines, because when you are at the borders, you can have side effects. In your case, you could start your selection in the middle of the first line and finish it to the middle of the last line.
          Finally, as it was doing something, I corrected the problem with a little special case.

          Raphaël Franchet added a comment - First of all, please note this is not the right way to make lists (The proof is that your way works only if the last line you select is not the last line in the para) Secondly, it is often a better thing to partially select lines, because when you are at the borders, you can have side effects. In your case, you could start your selection in the middle of the first line and finish it to the middle of the last line. Finally, as it was doing something, I corrected the problem with a little special case.

            raphael Raphaël Franchet
            joel Joel Vimenet (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Created:
              Updated:
              Resolved: