Skip to content

Commit

Permalink
Fixed mediapicker field inside lists not properly updating the value …
Browse files Browse the repository at this point in the history
…on the target input (fixes #1157)
  • Loading branch information
w00fz committed Jul 17, 2017
1 parent a7a8b7f commit 316008f
Show file tree
Hide file tree
Showing 4 changed files with 33 additions and 31 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@

1. [](#bugfix)
* Fixes issue when saving pages without a `folder` element [#1163](https://github.com/getgrav/grav-plugin-admin/issues/1163)
* Fixed mediapicker field inside lists not properly updating the value on the target input [#1157](https://github.com/getgrav/grav-plugin-admin/issues/1157)

# v1.5.0
## 07/16/2017
Expand Down
7 changes: 4 additions & 3 deletions themes/grav/app/forms/fields/mediapicker.js
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,8 @@ $(function() {
}, 10);
});

body.on('click', '[data-mediapicker-modal-trigger]', function() {
body.on('click', '[data-mediapicker-modal-trigger]', function(event) {
const element = $(event.currentTarget);
let modal_identifier = $(this).data('grav-mediapicker-unique-identifier');
let modal_element = body.find(`[data-remodal-unique-identifier="${modal_identifier}"]`);
modal = $.remodal.lookup[modal_element.data('remodal')];
Expand All @@ -35,6 +36,7 @@ $(function() {
}

modal.open();
modal.dataField = element.find('input');

// load all media
modal_element.find('.js__files').trigger('fillView');
Expand All @@ -48,8 +50,7 @@ $(function() {
let val = $(event.target).parents('.js__media-element').data('file-url');
let string = val.replace(/ /g, '%20');

let modal_identifier = $(event.target).parents('[data-remodal-mediapicker]').data('remodal-unique-identifier');
body.find('[data-grav-mediapicker-unique-identifier="' + modal_identifier + '"] input').val(string);
modal.dataField.val(string);

modal.close();
});
Expand Down
26 changes: 13 additions & 13 deletions themes/grav/js/admin.min.js

Large diffs are not rendered by default.

30 changes: 15 additions & 15 deletions themes/grav/js/vendor.min.js

Large diffs are not rendered by default.

0 comments on commit 316008f

Please sign in to comment.