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

Warning popup about sanitized HTML should appear BEFORE saving it and not after.

    XMLWordPrintable

    Details

    • Type: Feature Request
    • Status: OPEN
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: 10.5, 11.0
    • Fix Version/s: None
    • Labels:
      None
    • Previous Issue Keys:
      SAM-2578

      Description

      This issue can't be currently tested in trunk because the warning popup is not working there right now, I think that maybe due to some regression of Morpheus. I have tested it in a 10.x but I'm sure that the issue happens in trunk too.

      The steps to reproduce the issue are:

      1) Create a new Pool and a question in it.
      2) In the CKEditor, in Source edition, put some HTML not allowed by AntiSamy filters by default, i.e:

      <img src="http://latex.codecogs.com/gif.latex?\frac{\mathrm

      {d}&space;35x}{\mathrm{d}

      &space;x%2A2}" title="\frac{\mathrm

      {d} 35x}{\mathrm{d}

      x%2A2}" />

      3) Save the question. The questions is saved and no warning is shown to users. So users think it is working fine.
      4) Edit the question again. The popup shows the warning (not very explanatory) and the bad HTML is deleted. Cancelling the edition does not preserve it, so the users have lost a part of their work without any chance of recovering it.

      IMHO the warning should be shown when user saves the work in step 3, not letting users to save the work until the conflicting HTML has been removed or changed.

        Gliffy Diagrams

          Zeplin

            Attachments

            1. SAM-2578-1.png
              SAM-2578-1.png
              106 kB
            2. SAM-2578-2.png
              SAM-2578-2.png
              90 kB
            3. SAM-2578-3.png
              SAM-2578-3.png
              121 kB

              Issue Links

                Activity

                  People

                  Assignee:
                  Unassigned Unassigned
                  Reporter:
                  daniel.merino Daniel Merino Echeverría
                  Votes:
                  0 Vote for this issue
                  Watchers:
                  11 Start watching this issue

                    Dates

                    Created:
                    Updated:

                      Git Integration