-
Notifications
You must be signed in to change notification settings - Fork 62
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
EZP-31602: Moving to trash confirmation modal is not updated after sub-items bulk move #1744
EZP-31602: Moving to trash confirmation modal is not updated after sub-items bulk move #1744
Conversation
…b-items bulk move
@@ -7,6 +7,7 @@ | |||
const sortContainer = doc.querySelector('[data-sort-field][data-sort-order]'); | |||
const sortField = sortContainer.getAttribute('data-sort-field'); | |||
const sortOrder = sortContainer.getAttribute('data-sort-order'); | |||
const contentName = doc.querySelector('.ez-sil').dataset.contentName; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't think this should be in the subitems div. This should be stored in the modal itself to be independent.
|
||
modalBody.innerHTML = message; | ||
modalSendToTrashButton.toggleAttribute('disabled', false); | ||
modalSendToTrashButton.classList.toggle('disabled', false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove instead of toggle
); | ||
|
||
modalBody.innerHTML = message; | ||
modalSendToTrashButton.toggleAttribute('disabled', false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removeattribute instead of toggleattribute
@@ -118,6 +118,36 @@ | |||
|
|||
$(SELECTOR_MODAL_BULK_ACTION_FAIL).modal('show'); | |||
}; | |||
const refreshTrashModal = (event) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder, maybe we should put these changes here:
https://github.com/ezsystems/ezplatform-admin-ui/blob/1.5/src/bundle/Resources/public/js/scripts/admin.trash.js
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
QA Approve - tested on 2.5
Tests on v3 after merge-up
@mnocon same here, it is failing. |
You can merge it up. |
Checklist:
$ composer fix-cs
)