Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-39309

CKEditor doesn't allow quotes in ALT TEXT

    XMLWordPrintable

    Details

    • Previous Issue Keys:
      SAK-33401, KNL-1549
    • Test Plan:
      Hide
      1. Login to a course.
      2. Edit a CKEditor (RTF) area
      3. Add an image (can be local or publicly available on the internet)
      4. Edit the alternative text field to include some text within quotes such as "Coin" - including quotes
      5. Save the RTF and look to see if Sakai indicates an error, some HTML code error in the top right....
      6. Next go back into editing the same RTF area - checking the image properties box - the quotes should remain - but seem to get stripped out.
        Expected behavior is that the quotes remain.
      Show
      Login to a course. Edit a CKEditor (RTF) area Add an image (can be local or publicly available on the internet) Edit the alternative text field to include some text within quotes such as "Coin" - including quotes Save the RTF and look to see if Sakai indicates an error, some HTML code error in the top right.... Next go back into editing the same RTF area - checking the image properties box - the quotes should remain - but seem to get stripped out. Expected behavior is that the quotes remain.

      Description

      When adding quotes with ALT TEXT in the CKEditor's image edit properties box - quotes are stripped out of the description.  Further, if the only ALT TEXT added is fully enclosed in quotes all of the ALT TEXT provided for the image is stripped out.  

      It's not clear if this is an ANTISAMY issue or if this is specific to the way the CKEditor works. In testing the CKEditor (latest: 4.7) at the demo at cksource - it doesn't strip out the "" for the alttext though the demo never compiles the content and displays it the same way as it's embedded in Sakai.

      See the issue demonstrated here:

      http://somup.com/cbQ3IUVDSj 

      Reported by user associated with sakai.lampschools.org 

        Gliffy Diagrams

          Zeplin

            Attachments

              Issue Links

                Activity

                  People

                  Assignee:
                  maintenanceteam Core Team
                  Reporter:
                  davideveland Dave Eveland
                  Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                    Dates

                    Created:
                    Updated:

                      Git Integration