[MGNLDIFF-108] "Show changes" feature quite unresponsive when title field has no text Created: 16/Jun/21  Updated: 21/Jun/23

Status: Open
Project: Magnolia Diff Module
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Bug Priority: Neutral
Reporter: Carlos Cantalapiedra Assignee: Unassigned
Resolution: Unresolved Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: PNG File span_diff.png     PNG File span_moved.png    
Issue Links:
Problem/Incident
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Epic Link: AuthorX Support
Team: AuthorX

 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


Generated at Mon Feb 12 05:21:27 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.