Skip to content

Commit 67eb6fe

Browse files
committed
Merge remote-tracking branch 'vanilla/MAGETWO-39669' into PR
2 parents 88fbfc7 + a3c1a97 commit 67eb6fe

File tree

2 files changed

+26
-23
lines changed

2 files changed

+26
-23
lines changed

app/code/Magento/Checkout/view/frontend/web/js/sidebar.js

Lines changed: 18 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -24,41 +24,41 @@ define([
2424
},
2525

2626
_initContent: function() {
27-
var self = this;
27+
var self = this,
28+
events = {};
2829

2930
this.element.decorate('list', this.options.isRecursive);
3031

31-
$(this.options.button.close).click(function(event) {
32+
events['click ' + this.options.button.close] = function(event) {
3233
event.stopPropagation();
3334
$(self.options.targetElement).dropdownDialog("close");
34-
});
35-
36-
$(this.options.button.checkout).on('click', $.proxy(function() {
35+
};
36+
events['click ' + this.options.button.checkout] = $.proxy(function() {
3737
var cart = customerData.get('cart'),
3838
customer = customerData.get('customer');
3939

4040
if (customer() == false && !cart().isGuestCheckoutAllowed) {
4141
authenticationPopup.showModal();
42+
4243
return false;
4344
}
4445
location.href = this.options.url.checkout;
45-
}, this));
46-
47-
$(this.options.button.remove).click(function(event) {
46+
}, this);
47+
events['click ' + this.options.button.remove] = function(event) {
4848
event.stopPropagation();
4949
if (confirm(self.options.confirmMessage)) {
50-
self._removeItem($(this));
50+
self._removeItem($(event.target));
5151
}
52-
});
53-
54-
$(this.options.item.qty).keyup(function() {
55-
self._showItemButton($(this));
56-
});
57-
$(this.options.item.button).click(function(event) {
52+
};
53+
events['keyup ' + this.options.item.qty] = function(event) {
54+
self._showItemButton($(event.target));
55+
};
56+
events['click ' + this.options.item.button] = function(event) {
5857
event.stopPropagation();
59-
self._updateItemQty($(this))
60-
});
58+
self._updateItemQty($(event.target));
59+
};
6160

61+
this._on(this.element, events);
6262
this._calcHeight();
6363
this._isOverflowed();
6464
},
@@ -122,10 +122,8 @@ define([
122122
* Update content after update qty
123123
*
124124
* @param elem
125-
* @param response
126-
* @private
127125
*/
128-
_updateItemQtyAfter: function(elem, response) {
126+
_updateItemQtyAfter: function(elem) {
129127
this._hideItemButton(elem);
130128
},
131129

app/code/Magento/Checkout/view/frontend/web/js/view/minicart.js

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,21 +7,24 @@ define([
77
'Magento_Customer/js/customer-data',
88
'jquery',
99
'ko',
10-
'mage/url'
10+
'mage/url',
11+
'sidebar'
1112
], function (Component, customerData, $, ko, url) {
1213
'use strict';
1314

1415
var sidebarInitialized = false;
16+
1517
url.setBaseUrl(window.checkout.baseUrl);
1618

1719
function initSidebar() {
1820
var minicart = $("[data-block='minicart']");
21+
1922
minicart.trigger('contentUpdated');
2023
if (sidebarInitialized) {
2124
return false;
2225
}
2326
sidebarInitialized = true;
24-
minicart.mage('sidebar', {
27+
minicart.sidebar({
2528
"targetElement": "div.block.block-minicart",
2629
"url": {
2730
"checkout": window.checkout.checkoutUrl,
@@ -64,12 +67,14 @@ define([
6467
});
6568
},
6669
initSidebar: ko.observable(initSidebar),
67-
closeSidebar: function(element) {
70+
closeSidebar: function() {
6871
var minicart = $('[data-block="minicart"]');
72+
6973
minicart.on('click', '[data-action="close"]', function(event) {
7074
event.stopPropagation();
7175
minicart.find('[data-role="dropdownDialog"]').dropdownDialog("close");
7276
});
77+
7378
return true;
7479
},
7580
getItemRenderer: function (productType) {

0 commit comments

Comments
 (0)