[Forwardport] Move buttons definition to separate file #15757
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Original Pull Request
#15194
When trying to add new buttons with new actions (besides
reset
,save
andsaveAndContinue
) to a UiComponent form in the backend, it is needed to extend upon at least 2 files:Magento_Ui/js/form/adapter
andMagento_Ui/js/form/form
. The second file is easily extendable using mixins, but the first one is not: Mainly because thebuttons
definition is based on a local variable. This PR fixes this by moving thebuttons
definition in a separate file.Description
This PR adds a new file
buttons.js
that is called for inadapter
so that someone could create a mixin like follows:RequireJS configuration:
And in
Foo_Bar/js/form/form-mixin
:Unfortunately, this PR does not deal with how to use
foobar
in the end, which is hugely complex.Manual testing scenarios
Save
andSave And Continue
are still working in the backendContribution checklist