Merge pull request #32 from andy-hook/master

Fix: Incorrect margin application
theKashey · Jun 6, 2022 · 5f2c981 · 5f2c981
2 parents 2eae724 + c42b51f
commit 5f2c981
Showing 1 changed file with 1 addition and 6 deletions.
diff --git a/src/component.tsx b/src/component.tsx
@@ -70,13 +70,8 @@ const getStyles = (
  * Removes page scrollbar and blocks page scroll when mounted
  */
 export const RemoveScrollBar: React.FC<BodyScroll> = (props) => {
- const [gap, setGap] = React.useState(getGapWidth(props.gapMode));
-
- React.useEffect(() => {
- setGap(getGapWidth(props.gapMode));
- }, [props.gapMode]);
-
  const { noRelative, noImportant, gapMode = 'margin' } = props;
+ const gap = React.useMemo(() => getGapWidth(gapMode), [gapMode]);
 
  return <Style styles={getStyles(gap, !noRelative, gapMode, !noImportant ? '!important' : '')} />;
 };