cleanup, naming

Author: Lukas742

packages/main/src/components/ObjectPage/index.tsx

@@ -109,7 +109,7 @@ const ObjectPage = forwardRef<ObjectPageDomRef, ObjectPagePropTypes>((props, ref
   const sections = mode === ObjectPageMode.IconTabBar ? currentTabModeSection : children;
   const scrollEndHandler = useOnScrollEnd({ objectPageRef, setTabSelectId });
   // only required for IconTabBar mode
-  const [userSectionChange, setUserSectionChange] = useState(false);
+  const [wasUserSectionChange, setWasUserSectionChange] = useState(false);
 
   useEffect(() => {
     const currentSection =
@@ -605,7 +605,7 @@ const ObjectPage = forwardRef<ObjectPageDomRef, ObjectPagePropTypes>((props, ref
     scrollTimeout,
     setSelectedSubSectionId,
     setTabSelectId,
-    setUserSectionChange,
+    setWasUserSectionChange,
   });
   const objectPageStyles: CSSProperties = {
     ...style,
@@ -765,15 +765,14 @@ const ObjectPage = forwardRef<ObjectPageDomRef, ObjectPagePropTypes>((props, ref
           className={classNames.content}
           ref={(node) => {
             if (node) {
-              if (mode === ObjectPageMode.IconTabBar && userSectionChange) {
-                console.log('focus');
+              if (mode === ObjectPageMode.IconTabBar && wasUserSectionChange) {
                 node.querySelector('[data-component-name="ObjectPageSection"]')?.focus({
                   preventScroll: true,
                 });
               }
               objectPageContentRef.current = node;
             }
-            setUserSectionChange(false);
+            setWasUserSectionChange(false);
           }}
           // prevent content scroll when elements outside the content are focused
           onFocus={() => {

packages/main/src/components/ObjectPage/useHandleTabSelect.ts

@@ -23,7 +23,7 @@ interface UseHandleTabSelectProps {
   scrollTimeout: RefObject<number>;
   setSelectedSubSectionId: Dispatch<SetStateAction<string>>;
   setTabSelectId: Dispatch<SetStateAction<string | null>>;
-  setUserSectionChange: Dispatch<SetStateAction<boolean>>;
+  setWasUserSectionChange: Dispatch<SetStateAction<boolean>>;
 }
 
 export const useHandleTabSelect = ({
@@ -42,7 +42,7 @@ export const useHandleTabSelect = ({
   scrollTimeout,
   setSelectedSubSectionId,
   setTabSelectId,
-  setUserSectionChange,
+  setWasUserSectionChange,
 }: UseHandleTabSelectProps) => {
   const [onSectionSelectedArgs, setOnSectionSelectedArgs] = useState<
     | false
@@ -109,7 +109,7 @@ export const useHandleTabSelect = ({
     }
 
     if (mode === ObjectPageMode.IconTabBar) {
-      setUserSectionChange(true);
+      setWasUserSectionChange(true);
     }
   };
   // effect required - if event is called in `handleTabItemSelect` it's invoked twice in StrictMode