Bug 1435105 - Rename 'shippingRequested' class to 'shipping-related' to make the name clearer as well as follow file conventions. r?mattn
MozReview-Commit-ID: 6Dgzt69fFFq
--- a/toolkit/components/payments/res/containers/payment-dialog.js
+++ b/toolkit/components/payments/res/containers/payment-dialog.js
@@ -27,17 +27,17 @@ class PaymentDialog extends PaymentState
this._payButton = contents.querySelector("#pay");
this._payButton.addEventListener("click", this);
this._viewAllButton = contents.querySelector("#view-all");
this._viewAllButton.addEventListener("click", this);
this._orderDetailsOverlay = contents.querySelector("#order-details-overlay");
this._shippingTypeLabel = contents.querySelector("#shipping-type-label");
- this._shippingRequestedEls = contents.querySelectorAll(".shippingRequested");
+ this._shippingRelatedEls = contents.querySelectorAll(".shipping-related");
this._errorText = contents.querySelector("#error-text");
this._disabledOverlay = contents.getElementById("disabled-overlay");
this.appendChild(contents);
super.connectedCallback();
}
@@ -186,17 +186,17 @@ class PaymentDialog extends PaymentState
let totalItem = paymentDetails.totalItem;
let totalAmountEl = this.querySelector("#total > currency-amount");
totalAmountEl.value = totalItem.amount.value;
totalAmountEl.currency = totalItem.amount.currency;
this._orderDetailsOverlay.hidden = !state.orderDetailsShowing;
this._errorText.textContent = paymentDetails.error;
let paymentOptions = request.paymentOptions;
- for (let element of this._shippingRequestedEls) {
+ for (let element of this._shippingRelatedEls) {
element.hidden = !paymentOptions.requestShipping;
}
let shippingType = paymentOptions.shippingType || "shipping";
this._shippingTypeLabel.querySelector("label").textContent =
this._shippingTypeLabel.dataset[shippingType + "AddressLabel"];
this._renderPayButton(state);
--- a/toolkit/components/payments/res/paymentRequest.xhtml
+++ b/toolkit/components/payments/res/paymentRequest.xhtml
@@ -62,24 +62,24 @@
</div>
</header>
<div id="main-container">
<section id="payment-summary">
<h1>&paymentSummaryTitle;</h1>
<section>
- <div class="shippingRequested"
+ <div class="shipping-related"
id="shipping-type-label"
data-shipping-address-label="&shippingAddressLabel;"
data-delivery-address-label="&deliveryAddressLabel;"
data-pickup-address-label="&pickupAddressLabel;"><label></label></div>
- <address-picker class="shippingRequested" selected-state-key="selectedShippingAddress"></address-picker>
- <div class="shippingRequested"><label>&shippingOptionsLabel;</label></div>
- <shipping-option-picker class="shippingRequested"></shipping-option-picker>
+ <address-picker class="shipping-related" selected-state-key="selectedShippingAddress"></address-picker>
+ <div class="shipping-related"><label>&shippingOptionsLabel;</label></div>
+ <shipping-option-picker class="shipping-related"></shipping-option-picker>
<div><label>&paymentMethodsLabel;</label></div>
<payment-method-picker selected-state-key="selectedPaymentCard"></payment-method-picker>
<div><label id="error-text"></label></div>
</section>
<footer id="controls-container">
<button id="cancel">&cancelPaymentButton.label;</button>
<button id="pay"