Problem/Motivation
When reviewing the diff for content that uses Paragraphs (or any other similar module) as components, it is difficult to understand what each field is associated with. One field could be repeated multiple times on the various referenced Paragraph entities, making the diff hard to understand.
Steps to reproduce
Review the diff for content that uses Paragraphs or any referenced entity.
Proposed resolution
Adjust the label to include relevant context.
Remaining tasks
- Confirm that we like the changes and there isn't a better approach.
- Add tests?
User interface changes
Changes the field label on the diff page.
Before:

After:

API changes
N/A
Data model changes
N/A
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 3487580-3-diff-bundle-index-label-wse.patch | 581 bytes | adamzimmermann |
| Screenshot 2024-11-14 at 11.14.12 AM.png | 200.87 KB | adamzimmermann | |
| Screenshot 2024-11-14 at 11.40.40 AM.png | 125.68 KB | adamzimmermann |
Issue fork diff-3487580
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #3
adamzimmermann commentedIf you are using the diff module with the Workspaces Extra module, you will need the patch I just uploaded to alter the service definition.
Comment #4
adamzimmermann commentedComment #5
acbramley commentedMR is failing, will need a rebase.
Comment #6
adamzimmermann commentedMerge conflict resolved, ready for further testing.
Comment #7
acbramley commentedThis is quite a good improvement, I've left some minor comments and I think it would be good to get some tests too.
Comment #8
adamzimmermann commentedChanges implemented, except for the tests. Let me know if the changes align with your vision.
Tests are something I'm not very proficient at. If someone else could pick that part up that would be great.
Comment #9
acbramley commentedLooking good, back to NW for tests