Uploaded image for project: 'PUBLIC - Liferay Portal Community Edition'
  1. PUBLIC - Liferay Portal Community Edition
  2. LPS-111213

Alloy Editor on Firefox: "link", "remove link" and "link browse" do not work when inside ul li tags with spaces

    Details

    • Branch Version/s:
      7.2.x, 7.1.x, 7.0.x
    • Backported to Branch:
      Committed
    • Fix Priority:
      3

      Description

      Reproduction steps:

      1) Patch 7.0 DXP with de-89 and start the server, login as Admin on Firefox
      2) Add a Web Content Display portlet on the Welcome page and click on the "+" button to create a new Basic Web Content
      3) Create a list with the second bulletpoint made of two words separated by a space, like this:

      <ul>
      	<li>oneword</li>
      	<li><a href="#go2" target="">two words</a></li>
      </ul>
      

      4) Select the "oneword" and insert a link to any URL, click on the checkmark to save the action:

      Expected:
      Link to be saved and "oneword" to have a valid hyperlink

      Actual behavior:
      Linking operation will work, but the console will print:

      IndexSizeError: Index or size is negative or greater than the allowed amount ckeditor.js
      

      5) Click again on "oneword" and then on the broken chain icon to remove the hyperlink:

      Expected:
      Link to be removed

      Actual behavior:
      Un-linking operation will fail and the console will print:

      IndexSizeError: Index or size is negative or greater than the allowed amount ckeditor.js
      

      6) Click again on the "oneword" link, then on the folder icon on the right to link it to a document
      7) Select any document from Liferay or local disk, then click on "Add":

      Expected:
      Linking operation to work

      Actual behavior:
      This error will be printed again in the console:

      IndexSizeError: Index or size is negative or greater than the allowed amount ckeditor.js
      
      TypeError: l is undefined
      

      and it will not be possible to close the pop-up.

       All above use-cases will not trigger any issues when the second word in the list has no spaces, e.g.:

      <ul>
      	<li><a href="#go1" target="">oneword</a></li>
      	<li><a href="#go2" target="">twowords</a></li>
      </ul>
      

       Reproduced on 7.0.x @ 1139e8971d3dab4681eb25eded6189f6d8e06422673fe7532bf891dad2bf7249
       Reproduced on master @ 5ba27fb882f181f7a595c01c6acac425e444ee19b0d782acb80b15c3128522d2

      Fixed on master on LPS-110922

       

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              summer.zhang Summer Zhang
              Reporter:
              roland.pakai Roland Pákai
              Participants of an Issue:
              Recent user:
              Clarissa Velazquez
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Days since last comment:
                29 weeks ago

                  Packages

                  Version Package
                  7.0.0 DXP FP91
                  7.0.X
                  7.1.10 DXP FP18
                  7.1.X
                  7.2.10 DXP FP5
                  7.2.10.2 DXP SP2
                  7.2.X