second proposed fix

Hacker0x01 · Oct 5, 2022 · dc1931a · dc1931a
1 parent 269f8ca
commit dc1931a
Showing 1 changed file with 9 additions and 7 deletions.
diff --git a/src/time.jsx b/src/time.jsx
@@ -28,7 +28,7 @@ export default class Time extends React.Component {
 
   static calcCenterPosition = (listHeight, centerLiRef) => {
     return (
-      centerLiRef?.offsetTop - (listHeight / 2 - centerLiRef?.clientHeight / 2)
+      centerLiRef.offsetTop - (listHeight / 2 - centerLiRef.clientHeight / 2)
     );
   };
 
@@ -62,12 +62,14 @@ export default class Time extends React.Component {
 
   componentDidMount() {
     // code to ensure selected time will always be in focus within time window when it first appears
-    this.list.scrollTop = Time.calcCenterPosition(
-      this.props.monthRef
-        ? this.props.monthRef.clientHeight - this.header.clientHeight
-        : this.list.clientHeight,
-      this.centerLi
-    );
+    this.list.scrollTop =
+      this.centerLi &&
+      Time.calcCenterPosition(
+        this.props.monthRef
+          ? this.props.monthRef.clientHeight - this.header.clientHeight
+          : this.list.clientHeight,
+        this.centerLi
+      );
     if (this.props.monthRef && this.header) {
       this.setState({
         height: this.props.monthRef.clientHeight - this.header.clientHeight,