Uploaded image for project: 'Magnolia Diff Module'
  1. Magnolia Diff Module
  2. MGNLDIFF-108

"Show changes" feature quite unresponsive when title field has no text

    XMLWordPrintable

Details

    • Bug
    • Resolution: Unresolved
    • Neutral
    • None
    • None
    • None

    Description

      Steps to reproduce

      1.  At our demo move to Configuration App and navigate to /modules/form/dialogs/formEdit/form/tabs/tabMain/fields/title
      2.  Switch required from true to false
      3. Publish /travel/contact (which will generate version 1.0 of this page)
      4. Edit /travel/contact page and remove the label for email field on the contact form
      5. Publish /travel/contact (which will generate version 1.1 of this page)
      6. Use "Compare versions" feature and use inspect Tool from browser to see the differences
      7. Check that a lot of <span>> tags with same id has been generated
      8. Check that the span tag which has been removed (title) is moved to other field (which is an error), check (span_moved.png)

      Expected results

      Span tags are generated on the proper place

      Actual results

      7 or more empty tags are created (check span_diff.png)

      The diffs (label removed) is shown on an incorrect place (on this example, label for email field is removed but it is shown on the next field -> Subject, which causes confusion to the editors)

      Workaround

      N/A

      Development notes

      Only reproducible for that required field, for other fields which are not requited (like description) the issue doesn't happen

      Checklists

        Acceptance criteria

        Attachments

          1. span_diff.png
            span_diff.png
            12 kB
          2. span_moved.png
            span_moved.png
            57 kB

          Activity

            People

              Unassigned Unassigned
              ccantalapiedra Carlos Cantalapiedra
              AuthorX
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:

                Checklists

                  Bug DoR
                  Task DoD