Skip to content

Conversation

@tmastalirsch-tlv
Copy link

  • Introduced a new copy button in the GenericMultiField interface.
  • Implemented the _copyItem method to handle item duplication.
  • Added CSS styles for the copy button.
  • Updated the crxPath retrieval logic to support data-formid attribute.
  • Ensured copy button is disabled in read-only mode.
Screenshot 2025-08-12 at 12 08 54 Screenshot 2025-08-12 at 12 09 06 Screenshot 2025-08-12 at 12 09 12

MichaelHeil and others added 5 commits March 22, 2024 10:09
- Introduced a new copy button in the GenericMultiField interface.
- Implemented the _copyItem method to handle item duplication.
- Added CSS styles for the copy button.
- Updated the crxPath retrieval logic to support data-formid attribute.
- Ensured copy button is disabled in read-only mode.
@tmastalirsch-tlv
Copy link
Author

Hey guys,

I have a client who wants to be able to copy individual items within a multifield, as he only needs to adjust small parts, such as a timestamp, in the subdialog, but the rest should remain the same, so I've added a copy function

Greats @tmastalirsch-tlv

@MichaelHeil
Copy link
Collaborator

Hello @tmastalirsch-tlv ,
this is awesome. Thank you for pushing back that feature and your contribution to this solution.
We will be running some internal tests on the feature. We will also need to make it a feature flag on the multifield dialog level in order to be able to switch this on and off for each instance (for automated ID handling, etc.).

Would you please be so kind and remove the change based on issue #69 in the CUI.GenericMultiField.js

In addition, could you please redirect the MR to the https://github.com/merkle-open/aem-generic-multifield/tree/develop branch?

best regards

@tmastalirsch-tlv tmastalirsch-tlv changed the base branch from master to develop August 13, 2025 09:22
tmastalirsch-tlv and others added 2 commits August 13, 2025 11:24
- Simplified the logic for obtaining the crxPath by removing support for the data-formid attribute.
- Ensured that the crxPath is now exclusively derived from the action attribute of the current form.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants