From 722dc1ab21363ee2eb584c382bdf1214c74197cf Mon Sep 17 00:00:00 2001
From: dannd <dannd96.it@gmail.com>
Date: Sat, 1 Feb 2020 11:41:37 +0700
Subject: [PATCH] fix: #105 fix review

---
 .../src/components/HelpGuide/index.tsx        | 29 +++++++++----------
 1 file changed, 14 insertions(+), 15 deletions(-)

diff --git a/packages/elements/src/components/HelpGuide/index.tsx b/packages/elements/src/components/HelpGuide/index.tsx
index c7740f64e3..fca9b9506a 100644
--- a/packages/elements/src/components/HelpGuide/index.tsx
+++ b/packages/elements/src/components/HelpGuide/index.tsx
@@ -40,14 +40,14 @@ export const caculateStepChange = ({
   helpguideRef: React.RefObject<HTMLElement>
   isMobileScreen: boolean
 }) => () => {
-  if (currentStepRef.current && wrapperStepRef.current && helpguideRef.current) {
-    if (isMobileScreen) {
-      helpguideRef.current.style.height = '100%'
-      wrapperStepRef.current.style.transform = `translate3d(-${currentStepRef.current.offsetLeft}px, 0, 0)`
-    } else {
-      helpguideRef.current.style.height = `${currentStepRef.current.offsetHeight}px`
-      wrapperStepRef.current.style.transform = `translate3d(0, -${currentStepRef.current.offsetTop}px, 0)`
-    }
+  if (currentStepRef.current && wrapperStepRef.current && helpguideRef.current && isMobileScreen) {
+    helpguideRef.current.style.height = '100%'
+    wrapperStepRef.current.style.transform = `translate3d(-${currentStepRef.current.offsetLeft}px, 0, 0)`
+  }
+
+  if (currentStepRef.current && wrapperStepRef.current && helpguideRef.current && !isMobileScreen) {
+    helpguideRef.current.style.height = `${currentStepRef.current.offsetHeight}px`
+    wrapperStepRef.current.style.transform = `translate3d(0, -${currentStepRef.current.offsetTop}px, 0)`
   }
 }
 
@@ -62,13 +62,12 @@ export const caculateWrapperWith = ({
   total: number
   isMobileScreen: boolean
 }) => () => {
-  if (helpguideRef.current && wrapperStepRef.current) {
-    if (isMobileScreen) {
-      const wrapperWidth = helpguideRef.current.clientWidth * total
-      wrapperStepRef.current.style.width = `${wrapperWidth}px`
-    } else {
-      wrapperStepRef.current.style.width = '100%'
-    }
+  if (helpguideRef.current && wrapperStepRef.current && isMobileScreen) {
+    const wrapperWidth = helpguideRef.current.clientWidth * total
+    wrapperStepRef.current.style.width = `${wrapperWidth}px`
+  }
+  if (helpguideRef.current && wrapperStepRef.current && !isMobileScreen) {
+    wrapperStepRef.current.style.width = '100%'
   }
 }