From 82cf9380a02cfb58934486e84982736e75b356ec Mon Sep 17 00:00:00 2001
From: Bartosz Herba <bherba@vuestorefront.io>
Date: Tue, 26 Jul 2022 09:17:21 +0200
Subject: [PATCH] refactor: remove style duplication in my account route
 components

---
 .../AddressesDetails/AddressEdit.vue          |  8 ++----
 .../MyAccount/AddressesDetails/AddressNew.vue |  7 +----
 .../AddressesDetails/AddressesDetails.vue     |  7 +----
 .../customer/pages/MyAccount/MyNewsletter.vue |  8 +-----
 .../customer/pages/MyAccount/MyReviews.vue    | 27 ++-----------------
 .../customer/pages/MyAccount/MyWishlist.vue   |  9 +------
 .../pages/MyAccount/ResetPassword.vue         |  4 +++
 .../modules/customer/pages/styles/shared.scss | 26 ++++++++++++++++++
 8 files changed, 38 insertions(+), 58 deletions(-)
 create mode 100644 packages/theme/modules/customer/pages/styles/shared.scss

diff --git a/packages/theme/modules/customer/pages/MyAccount/AddressesDetails/AddressEdit.vue b/packages/theme/modules/customer/pages/MyAccount/AddressesDetails/AddressEdit.vue
index a7df3bbed..2a93eefc0 100644
--- a/packages/theme/modules/customer/pages/MyAccount/AddressesDetails/AddressEdit.vue
+++ b/packages/theme/modules/customer/pages/MyAccount/AddressesDetails/AddressEdit.vue
@@ -69,10 +69,6 @@ export default defineComponent({
 </script>
 
 <style lang="scss" scoped>
-.message {
-  font-family: var(--font-family--primary);
-  line-height: 1.6;
-  font-size: var(--font-size--base);
-  margin: 0 0 var(--spacer-base);
-}
+@import "../../styles/shared.scss";
+
 </style>
diff --git a/packages/theme/modules/customer/pages/MyAccount/AddressesDetails/AddressNew.vue b/packages/theme/modules/customer/pages/MyAccount/AddressesDetails/AddressNew.vue
index 44fc40e46..b7f0cd0b6 100644
--- a/packages/theme/modules/customer/pages/MyAccount/AddressesDetails/AddressNew.vue
+++ b/packages/theme/modules/customer/pages/MyAccount/AddressesDetails/AddressNew.vue
@@ -43,10 +43,5 @@ export default defineComponent({
 </script>
 
 <style lang="scss" scoped>
-.message {
-  font-family: var(--font-family--primary);
-  line-height: 1.6;
-  font-size: var(--font-size--base);
-  margin: 0 0 var(--spacer-base);
-}
+@import "../../styles/shared.scss";
 </style>
diff --git a/packages/theme/modules/customer/pages/MyAccount/AddressesDetails/AddressesDetails.vue b/packages/theme/modules/customer/pages/MyAccount/AddressesDetails/AddressesDetails.vue
index f5efb8ccc..f50f59019 100644
--- a/packages/theme/modules/customer/pages/MyAccount/AddressesDetails/AddressesDetails.vue
+++ b/packages/theme/modules/customer/pages/MyAccount/AddressesDetails/AddressesDetails.vue
@@ -179,12 +179,7 @@ export default defineComponent({
 </script>
 
 <style lang="scss" scoped>
-.message {
-  font-family: var(--font-family--primary);
-  line-height: 1.6;
-  font-size: var(--font-size--base);
-  margin: 0 0 var(--spacer-base);
-}
+@import "../../styles/shared.scss";
 
 .addresses-list {
   margin-bottom: var(--spacer-base);
diff --git a/packages/theme/modules/customer/pages/MyAccount/MyNewsletter.vue b/packages/theme/modules/customer/pages/MyAccount/MyNewsletter.vue
index ec7e355c7..99f96dc18 100644
--- a/packages/theme/modules/customer/pages/MyAccount/MyNewsletter.vue
+++ b/packages/theme/modules/customer/pages/MyAccount/MyNewsletter.vue
@@ -82,13 +82,7 @@ export default defineComponent({
 </script>
 
 <style lang='scss' scoped>
-.tab-orphan {
-  @include for-mobile {
-    --tabs-title-display: none;
-    --tabs-content-padding: 0;
-    --tabs-conent-border-width: 0;
-  }
-}
+@import '../styles/shared.scss';
 
 .form {
   &__element {
diff --git a/packages/theme/modules/customer/pages/MyAccount/MyReviews.vue b/packages/theme/modules/customer/pages/MyAccount/MyReviews.vue
index 2e4412ef5..a6538030e 100644
--- a/packages/theme/modules/customer/pages/MyAccount/MyReviews.vue
+++ b/packages/theme/modules/customer/pages/MyAccount/MyReviews.vue
@@ -90,6 +90,8 @@ export default defineComponent({
 </script>
 
 <style lang="scss" scoped>
+@import '../styles/shared.scss';
+
 .no-reviews {
   &__title {
     margin: 0 0 var(--spacer-lg) 0;
@@ -98,31 +100,6 @@ export default defineComponent({
   }
 }
 
-.tab-orphan {
-  @include for-mobile {
-    --tabs-title-display: none;
-    --tabs-content-padding: 0;
-    --tabs-conent-border-width: 0;
-  }
-}
-
-.message {
-  margin: 0 0 var(--spacer-2xl) 0;
-  font: var(--font-weight--light) var(--font-size--base) / 1.6
-    var(--font-family--primary);
-
-  &__link {
-    color: var(--c-primary);
-    --link-weight: var(--font-weight--medium);
-    --link-font-family: var(--font-family--primary);
-    --link-font-size: var(--font-size--base);
-    text-decoration: none;
-
-    &:hover {
-      color: var(--c-text);
-    }
-  }
-}
 .sf-review__rating {
   width: 100%;
   display: flex;
diff --git a/packages/theme/modules/customer/pages/MyAccount/MyWishlist.vue b/packages/theme/modules/customer/pages/MyAccount/MyWishlist.vue
index d49f62185..6e88d2cb1 100644
--- a/packages/theme/modules/customer/pages/MyAccount/MyWishlist.vue
+++ b/packages/theme/modules/customer/pages/MyAccount/MyWishlist.vue
@@ -251,13 +251,7 @@ export default defineComponent({
 </script>
 
 <style lang="scss" scoped>
-.tab-orphan {
-  @include for-mobile {
-    --tabs-title-display: none;
-    --tabs-content-padding: 0;
-    --tabs-conent-border-width: 0;
-  }
-}
+@import '../styles/shared.scss';
 
 .main {
   &.section {
@@ -459,7 +453,6 @@ export default defineComponent({
   }
 
   &__product-card {
-    --product-card-title-margin: var(--spacer-base) 0 0 0;
     --product-card-title-font-weight: var(--font-weight--medium);
     --product-card-title-margin: var(--spacer-xs) 0 0 0;
     flex: 1 1 50%;
diff --git a/packages/theme/modules/customer/pages/MyAccount/ResetPassword.vue b/packages/theme/modules/customer/pages/MyAccount/ResetPassword.vue
index 653be0599..415f8f915 100644
--- a/packages/theme/modules/customer/pages/MyAccount/ResetPassword.vue
+++ b/packages/theme/modules/customer/pages/MyAccount/ResetPassword.vue
@@ -232,6 +232,7 @@ export default defineComponent({
     margin: 0 0 var(--spacer-xl) 0;
   }
 }
+
 .action {
   display: flex;
   align-items: center;
@@ -242,12 +243,15 @@ export default defineComponent({
     margin: 0 0 0 var(--spacer-xs);
   }
 }
+
 .action {
   margin: var(--spacer-xl) 0 var(--spacer-xl) 0;
 }
+
 .checkbox {
   margin-bottom: var(--spacer-2xl);
 }
+
 .bottom {
   text-align: center;
   margin-bottom: var(--spacer-lg);
diff --git a/packages/theme/modules/customer/pages/styles/shared.scss b/packages/theme/modules/customer/pages/styles/shared.scss
new file mode 100644
index 000000000..b907947c9
--- /dev/null
+++ b/packages/theme/modules/customer/pages/styles/shared.scss
@@ -0,0 +1,26 @@
+.tab-orphan {
+  @include for-mobile {
+    --tabs-title-display: none;
+    --tabs-content-padding: 0;
+    --tabs-conent-border-width: 0;
+  }
+}
+
+.message {
+  font-family: var(--font-family--primary);
+  line-height: 1.6;
+  font-size: var(--font-size--base);
+  margin: 0 0 var(--spacer-base);
+
+  &__link {
+    color: var(--c-primary);
+    --link-weight: var(--font-weight--medium);
+    --link-font-family: var(--font-family--primary);
+    --link-font-size: var(--font-size--base);
+    text-decoration: none;
+
+    &:hover {
+      color: var(--c-text);
+    }
+  }
+}